fix: remove noisy logs (#290)

daniel-sanche · daniel-sanche · commit bdf827358de5 · 2021-05-12T10:58:05.000-07:00
diff --git a/google/cloud/logging_v2/handlers/handlers.py b/google/cloud/logging_v2/handlers/handlers.py
@@ -24,7 +24,14 @@
 
 DEFAULT_LOGGER_NAME = "python"
 
-EXCLUDED_LOGGER_DEFAULTS = ("google.cloud", "google.auth", "google_auth_httplib2")
+"""Exclude internal logs from propagating through handlers"""
+EXCLUDED_LOGGER_DEFAULTS = (
+    "google.cloud",
+    "google.auth",
+    "google_auth_httplib2",
+    "google.api_core.bidi",
+    "werkzeug",
+)
 
 _CLEAR_HANDLER_RESOURCE_TYPES = ("gae_app", "cloud_function")
 
@@ -221,6 +228,6 @@ def setup_logging(
     logger.setLevel(log_level)
     logger.addHandler(handler)
     for logger_name in all_excluded_loggers:
+        # prevent excluded loggers from propagating logs to handler
         logger = logging.getLogger(logger_name)
         logger.propagate = False
-        logger.addHandler(logging.StreamHandler())
diff --git a/tests/environment b/tests/environment
@@ -1 +1 @@
-Subproject commit df1b7c131575f8eb59120cef75709496602b7665
+Subproject commit f1937814bf78953a160fa33600f6af8cfdb82527
diff --git a/tests/unit/test_client.py b/tests/unit/test_client.py
@@ -799,7 +799,13 @@ def test_setup_logging(self):
         handler.transport.worker.stop()
 
         expected_kwargs = {
-            "excluded_loggers": ("google.cloud", "google.auth", "google_auth_httplib2"),
+            "excluded_loggers": (
+                "google.cloud",
+                "google.auth",
+                "google_auth_httplib2",
+                "google.api_core.bidi",
+                "werkzeug",
+            ),
             "log_level": 20,
         }
         self.assertEqual(kwargs, expected_kwargs)
@@ -836,7 +842,13 @@ def test_setup_logging_w_extra_kwargs(self):
         handler.transport.worker.stop()
 
         expected_kwargs = {
-            "excluded_loggers": ("google.cloud", "google.auth", "google_auth_httplib2"),
+            "excluded_loggers": (
+                "google.cloud",
+                "google.auth",
+                "google_auth_httplib2",
+                "google.api_core.bidi",
+                "werkzeug",
+            ),
             "log_level": 20,
         }
         self.assertEqual(kwargs, expected_kwargs)
