From 2f9fd5ab00fa915fd76e5ebcfbbe458b17d0168e Mon Sep 17 00:00:00 2001
From: Erik Johnston <erik@matrix.org>
Date: Fri, 7 Aug 2020 14:53:05 +0100
Subject: [PATCH] Don't log OPTIONS request at INFO (#8049)

---
 changelog.d/8049.misc | 1 +
 synapse/http/site.py  | 8 +++++++-
 2 files changed, 8 insertions(+), 1 deletion(-)
 create mode 100644 changelog.d/8049.misc

diff --git a/changelog.d/8049.misc b/changelog.d/8049.misc
new file mode 100644
index 000000000..7fce36215
--- /dev/null
+++ b/changelog.d/8049.misc
@@ -0,0 +1 @@
+Log `OPTIONS` requests at `DEBUG` rather than `INFO` level to reduce amount logged at `INFO`.
diff --git a/synapse/http/site.py b/synapse/http/site.py
index 79a9229a2..6e79b4782 100644
--- a/synapse/http/site.py
+++ b/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):