Browse Source

Don't log OPTIONS request at INFO (#8049)

Erik Johnston 3 years ago
parent
commit
2f9fd5ab00
2 changed files with 8 additions and 1 deletions
  1. 1 0
      changelog.d/8049.misc
  2. 7 1
      synapse/http/site.py

+ 1 - 0
changelog.d/8049.misc

@@ -0,0 +1 @@
+Log `OPTIONS` requests at `DEBUG` rather than `INFO` level to reduce amount logged at `INFO`.

+ 7 - 1
synapse/http/site.py

@@ -319,7 +319,13 @@ class SynapseRequest(Request):
     def _should_log_request(self) -> bool:
         """Whether we should log at INFO that we processed the request.
         """
-        return self.path != b"/health"
+        if self.path == b"/health":
+            return False
+
+        if self.method == b"OPTIONS":
+            return False
+
+        return True
 
 
 class XForwardedForRequest(SynapseRequest):