Review comments

docs/sample_log_config.yaml

@@ -23,15 +23,14 @@ handlers:
         filename: /var/log/matrix-synapse/homeserver.log
         when: midnight
         backupCount: 3  # Does not include the current log file.
-        filters: [context]
         encoding: utf8
 
     # Default to buffering writes to log file for efficiency. This means that
     # will be a delay for WARNING/INFO/DEBUG logs to get written, but ERROR
     # logs will still be flushed immediately.
     buffer:
         class: logging.handlers.MemoryHandler
-        formatter: precise
+        filters: [context]
         target: file
         capacity: 100
 
@@ -46,8 +45,14 @@ loggers:
         # information such as access tokens.
         level: INFO
 
+    twisted:
+        # We send the twisted logging directly to the file handler,
+        # to work around https://github.com/matrix-org/synapse/issues/3471
+        handlers: [file]
+        propagate: false
+
 root:
     level: INFO
-    handlers: [buffer, console]
+    handlers: [buffer]
 
 disable_existing_loggers: false

synapse/config/logger.py

@@ -67,15 +67,14 @@
         filename: ${log_file}
         when: midnight
         backupCount: 3  # Does not include the current log file.
-        filters: [context]
         encoding: utf8
 
     # Default to buffering writes to log file for efficiency. This means that
     # will be a delay for WARNING/INFO/DEBUG logs to get written, but ERROR
     # logs will still be flushed immediately.
     buffer:
         class: logging.handlers.MemoryHandler
-        formatter: precise
+        filters: [context]
         target: file
         capacity: 100
 
@@ -90,9 +89,15 @@
         # information such as access tokens.
         level: INFO
 
+    twisted:
+        # We send the twisted logging directly to the file handler,
+        # to work around https://github.com/matrix-org/synapse/issues/3471
+        handlers: [file]
+        propagate: false
+
 root:
     level: INFO
-    handlers: [buffer, console]
+    handlers: [buffer]
 
 disable_existing_loggers: false
 """