From bdf827358de5935f736ecd73ab10b2d861daf690 Mon Sep 17 00:00:00 2001
From: Daniel Sanche <sanche@google.com>
Date: Tue, 11 May 2021 14:11:49 -0700
Subject: [PATCH] fix: remove noisy logs (#290)

---
 google/cloud/logging_v2/handlers/handlers.py | 11 +++++++++--
 tests/environment                            |  2 +-
 tests/unit/test_client.py                    | 16 ++++++++++++++--
 3 files changed, 24 insertions(+), 5 deletions(-)

diff --git a/google/cloud/logging_v2/handlers/handlers.py b/google/cloud/logging_v2/handlers/handlers.py
index 97332142..3580a4e9 100644
--- 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
index df1b7c13..f1937814 160000
--- 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
index f33f1cbd..9dbfa87f 100644
--- 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)