From 1cc2bd55f815441d1f719b5872bf4e8f12dcbb07 Mon Sep 17 00:00:00 2001
From: matthewswspence <Matthewswspence@gmail.com>
Date: Mon, 6 Jan 2025 15:40:03 -0600
Subject: [PATCH] test logging without json

---
 src/registrar/config/settings.py | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/registrar/config/settings.py b/src/registrar/config/settings.py
index b6b87c6eb..e0d32274d 100644
--- a/src/registrar/config/settings.py
+++ b/src/registrar/config/settings.py
@@ -519,13 +519,16 @@ LOGGING = {
             "level": env_log_level,
             "class": "logging.StreamHandler",
             "formatter": "verbose",
-            "filters": ["below_error"],
+            # "filters": ["below_error"],
         },
         "django.server": {
             "level": "INFO",
             "class": "logging.StreamHandler",
             "formatter": "django.server",
         },
+        # log all messages at ERROR level or higher using json formatter
+        # We do this because error logs often comprise many lines,
+        # and json formatting makes them easier to parse.
         "json": {
             "level": "ERROR",
             "class": "logging.StreamHandler",
@@ -535,7 +538,9 @@ LOGGING = {
         # because containerized apps
         # do not log to the file system.
     },
+    # filters are used to filter messages based on a callback function
     "filters": {
+        # filter for messages below ERROR level
         "below_error": {
             "()": "django.utils.log.CallbackFilter",
             "callback": lambda record: record.levelno < logging.ERROR,
@@ -546,7 +551,7 @@ LOGGING = {
     "loggers": {
         # Django's generic logger
         "django": {
-            "handlers": ["console", "json"],
+            "handlers": ["console"],
             "level": "INFO",
             "propagate": False,
         },