summary refs log tree commit diff
path: root/synapse/http/client.py
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2020-08-11 18:10:07 +0100
committerGitHub <noreply@github.com>2020-08-11 18:10:07 +0100
commita0f574f3c291374c11b7b1cac8fc7c4fbb6ef7b2 (patch)
tree3e945ba7857c36d57b5e8322cfe299efd2c319ba /synapse/http/client.py
parentChange the default log config to reduce disk I/O and storage (#8040) (diff)
downloadsynapse-a0f574f3c291374c11b7b1cac8fc7c4fbb6ef7b2.tar.xz
Reduce INFO logging (#8050)
c.f. #8021 

A lot of the code here is to change the `Completed 200 OK` logging to include the request URI so that we can drop the `Sending request...` log line.

Some notes:

1. We won't log retries, which may be confusing considering the time taken log line includes retries and sleeps.
2. The `_send_request_with_optional_trailing_slash` will always be logged *without* the forward slash, even if it succeeded only with the forward slash.
Diffstat (limited to 'synapse/http/client.py')
-rw-r--r--synapse/http/client.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/synapse/http/client.py b/synapse/http/client.py
index 529532a063..8aeb70cdec 100644
--- a/synapse/http/client.py
+++ b/synapse/http/client.py
@@ -297,7 +297,7 @@ class SimpleHttpClient(object):
         outgoing_requests_counter.labels(method).inc()
 
         # log request but strip `access_token` (AS requests for example include this)
-        logger.info("Sending request %s %s", method, redact_uri(uri))
+        logger.debug("Sending request %s %s", method, redact_uri(uri))
 
         with start_active_span(
             "outgoing-client-request",