From c8d953acc162f7a81c0fe664d57e2aed54e4653b Mon Sep 17 00:00:00 2001
From: Alex Kotov <kotovalexarian@gmail.com>
Date: Wed, 15 Jul 2020 08:34:29 +0500
Subject: [PATCH] Disable file logging for Matrix Synapse

---
 roles/matrix/tasks/common.yml                 |  1 -
 roles/matrix/tasks/synapse.yml                |  1 -
 roles/matrix/templates/synapse/log_config.yml | 10 +---------
 roles/matrix/vars/main.yml                    |  3 ---
 4 files changed, 1 insertion(+), 14 deletions(-)

diff --git a/roles/matrix/tasks/common.yml b/roles/matrix/tasks/common.yml
index 5531ad7..9d3c8cd 100644
--- a/roles/matrix/tasks/common.yml
+++ b/roles/matrix/tasks/common.yml
@@ -10,7 +10,6 @@
     - '{{ matrix__conf_dir }}'
     - '{{ matrix__opt_dir }}'
     - '{{ matrix__lib_dir }}'
-    - '{{ matrix__log_dir }}'
     - '{{ matrix__run_dir }}'
 
 - name: Recreate Matrix rundirs
diff --git a/roles/matrix/tasks/synapse.yml b/roles/matrix/tasks/synapse.yml
index 807074d..f9c87ba 100644
--- a/roles/matrix/tasks/synapse.yml
+++ b/roles/matrix/tasks/synapse.yml
@@ -43,7 +43,6 @@
     - '{{ matrix__synapse__conf_subdir }}'
     - '{{ matrix__synapse__opt_dir }}'
     - '{{ matrix__synapse__lib_dir }}'
-    - '{{ matrix__synapse__log_dir }}'
     - '{{ matrix__synapse__run_dir }}'
   notify: Load, enable and restart Matrix Synapse
 
diff --git a/roles/matrix/templates/synapse/log_config.yml b/roles/matrix/templates/synapse/log_config.yml
index 310295b..e6618e1 100644
--- a/roles/matrix/templates/synapse/log_config.yml
+++ b/roles/matrix/templates/synapse/log_config.yml
@@ -17,14 +17,6 @@ filters:
     request: ''
 
 handlers:
-  file:
-    class: logging.handlers.RotatingFileHandler
-    formatter: precise
-    filename: '{{ matrix__synapse__log_file }}'
-    maxBytes: 104857600
-    backupCount: 10
-    filters: [context]
-    encoding: utf8
   console:
     class: logging.StreamHandler
     formatter: precise
@@ -38,6 +30,6 @@ loggers:
 
 root:
   level: INFO
-  handlers: [file, console]
+  handlers: [console]
 
 disable_existing_loggers: false
diff --git a/roles/matrix/vars/main.yml b/roles/matrix/vars/main.yml
index 4ae3d7e..6db04fd 100644
--- a/roles/matrix/vars/main.yml
+++ b/roles/matrix/vars/main.yml
@@ -10,13 +10,11 @@ matrix__static__service: 'matrix-static'
 matrix__conf_dir: '/etc/matrix'
 matrix__opt_dir:  '/opt/matrix'
 matrix__lib_dir:  '/var/lib/matrix'
-matrix__log_dir:  '/var/log/matrix'
 matrix__run_dir:  '/var/run/matrix'
 
 matrix__synapse__conf_dir: '{{ matrix__conf_dir }}/synapse'
 matrix__synapse__opt_dir:  '{{ matrix__opt_dir  }}/synapse'
 matrix__synapse__lib_dir:  '{{ matrix__lib_dir  }}/synapse'
-matrix__synapse__log_dir:  '{{ matrix__log_dir  }}/synapse'
 matrix__synapse__run_dir:  '{{ matrix__run_dir  }}/synapse'
 
 matrix__static__conf_dir: '{{ matrix__conf_dir }}/static'
@@ -28,7 +26,6 @@ matrix__synapse__key_file:      '{{ matrix__synapse__conf_dir }}/signing_key'
 matrix__synapse__venv_dir:      '{{ matrix__synapse__opt_dir  }}/venv'
 matrix__synapse__media_dir:     '{{ matrix__synapse__lib_dir  }}/media_store'
 matrix__synapse__db_file:       '{{ matrix__synapse__lib_dir  }}/homeserver.db'
-matrix__synapse__log_file:      '{{ matrix__synapse__log_dir  }}/homeserver.log'
 matrix__synapse__pid_file:      '{{ matrix__synapse__run_dir  }}/homeserver.pid'
 
 matrix__static__conf_file:    '{{ matrix__static__conf_dir }}/config.json'