summary refs log tree commit diff
diff options
context:
space:
mode:
authorSergio Miguéns <lonyelon@gmail.com>2021-05-24 21:23:54 +0200
committerGitHub <noreply@github.com>2021-05-24 20:23:54 +0100
commit22a8838f626834c4ffc09761f4c5d65215cfc885 (patch)
tree9639cb2f8eb469bf8b40d5d867f75078b47ed6a3
parentRemove tls_fingerprints option (#9280) (diff)
downloadsynapse-22a8838f626834c4ffc09761f4c5d65215cfc885.tar.xz
Fix docker image to not log at `/homeserver.log` (#10045)
Fixes #9970

Signed-off-by: Sergio Miguéns Iglesias lonyelon@lony.xyz
-rw-r--r--changelog.d/10045.docker1
-rw-r--r--docker/conf/log.config4
2 files changed, 4 insertions, 1 deletions
diff --git a/changelog.d/10045.docker b/changelog.d/10045.docker
new file mode 100644
index 0000000000..70b65b0a01
--- /dev/null
+++ b/changelog.d/10045.docker
@@ -0,0 +1 @@
+Fix bug introduced in Synapse 1.33.0 which caused a `Permission denied: '/homeserver.log'` error when starting Synapse with the generated log configuration. Contributed by Sergio Miguéns Iglesias.
diff --git a/docker/conf/log.config b/docker/conf/log.config
index 34572bc0f3..a994626926 100644
--- a/docker/conf/log.config
+++ b/docker/conf/log.config
@@ -9,10 +9,11 @@ formatters:
 {% endif %}
 
 handlers:
+{% if LOG_FILE_PATH %}
   file:
     class: logging.handlers.TimedRotatingFileHandler
     formatter: precise
-    filename: {{ LOG_FILE_PATH or "homeserver.log" }}
+    filename: {{ LOG_FILE_PATH }}
     when: "midnight"
     backupCount: 6  # Does not include the current log file.
     encoding: utf8
@@ -29,6 +30,7 @@ handlers:
     # be written to disk.
     capacity: 10
     flushLevel: 30  # Flush for WARNING logs as well
+{% endif %}
 
   console:
     class: logging.StreamHandler