fix: disable logging response body for async logs (#1756)

* fix: disable logging response body for async logs

* remove unused test code

---------

Co-authored-by: Anthonios Partheniou <[email protected]>

Author: ohmayr

google/auth/aio/_helpers.py

@@ -53,5 +53,10 @@ async def response_log_async(logger: logging.Logger, response: Any) -> None:
         response: The HTTP response object to log.
     """
     if _helpers.is_logging_enabled(logger):
-        json_response = await _parse_response_async(response)
+        # TODO(https://github.com/googleapis/google-auth-library-python/issues/1755):
+        # Parsing the response for async streaming logging results in
+        # the stream to be empty downstream. For now, we will not be logging
+        # the response for async responses until we investigate further.
+        # json_response = await _parse_response_async(response)
+        json_response = None
         _helpers._response_log_base(logger, json_response)

tests/aio/test__helpers.py

@@ -65,17 +65,13 @@ async def test_response_log_debug_disabled(logger, caplog, base_logger):
 
 @pytest.mark.asyncio
 async def test_response_log_debug_enabled_response_json(logger, caplog, base_logger):
-    class MockResponse:
-        async def json(self):
-            return {"key1": "value1", "key2": "value2", "key3": "value3"}
-
-    response = MockResponse()
+    response = None
     caplog.set_level(logging.DEBUG, logger=_MOCK_CHILD_LOGGER_NAME)
     await _helpers.response_log_async(logger, response)
     assert len(caplog.records) == 1
     record = caplog.records[0]
     assert record.message == "Response received..."
-    assert record.httpResponse == {"key1": "value1", "key2": "value2", "key3": "value3"}
+    assert record.httpResponse == "<class 'NoneType'>"
 
 
 @pytest.mark.asyncio