From 41ec64e26ebb66b638c2b6bc06b8593339ad38a3 Mon Sep 17 00:00:00 2001
From: Jean-Marie Mineau <jean-marie.mineau@protonmail.com>
Date: Sat, 2 Oct 2021 21:07:47 +0200
Subject: [PATCH] move logs to /var

---
 roles/matrix-bridge-discord/tasks/main.yml         | 8 ++++++++
 roles/matrix-bridge-discord/templates/config.yaml  | 6 +++---
 roles/matrix-bridge-facebook/tasks/main.yml        | 8 ++++++++
 roles/matrix-bridge-facebook/templates/config.yaml | 2 +-
 4 files changed, 20 insertions(+), 4 deletions(-)

diff --git a/roles/matrix-bridge-discord/tasks/main.yml b/roles/matrix-bridge-discord/tasks/main.yml
index 7492295..7971d1f 100644
--- a/roles/matrix-bridge-discord/tasks/main.yml
+++ b/roles/matrix-bridge-discord/tasks/main.yml
@@ -89,6 +89,14 @@
     dest: /etc/systemd/system/bridge_discord.service
   notify: Restart the discord bridge
 
+- name: Ensure the log directory exists
+  file:
+    path: /var/log/bridge_discord
+    state: directory
+    group: root
+    owner: root
+    mode: u=rwx,g=rx,o=
+  
 - name: Enable the daemon
   systemd:
     name: bridge_discord
diff --git a/roles/matrix-bridge-discord/templates/config.yaml b/roles/matrix-bridge-discord/templates/config.yaml
index 90ecbac..a8bac46 100644
--- a/roles/matrix-bridge-discord/templates/config.yaml
+++ b/roles/matrix-bridge-discord/templates/config.yaml
@@ -49,12 +49,12 @@ logging:
   console: "warn" #silly, verbose, info, http, warn, error, silent
   lineDateFormat: "MMM-D HH:mm:ss.SSS" # This is in moment.js format
   files:
-    - file: "debug.log"
+    - file: "/var/log/bridge_discord/debug.log"
       disable:
         - "PresenceHandler" # Will not capture presence logging
-    - file: "warn.log" # Will capture warnings
+    - file: "/var/log/bridge_discord/warn.log" # Will capture warnings
       level: "warn"
-    - file: "botlogs.log" # Will capture logs from DiscordBot
+    - file: "/var/log/bridge_discord/botlogs.log" # Will capture logs from DiscordBot
       level: "info"
       enable:
         - "DiscordBot"
diff --git a/roles/matrix-bridge-facebook/tasks/main.yml b/roles/matrix-bridge-facebook/tasks/main.yml
index d398496..4a4decf 100644
--- a/roles/matrix-bridge-facebook/tasks/main.yml
+++ b/roles/matrix-bridge-facebook/tasks/main.yml
@@ -68,6 +68,14 @@
     dest: /etc/matrix-synapse/facebook-registration.yaml
     state: link
 
+- name: Ensure the log directory exists
+  file:
+    path: /var/log/bridge_facebook
+    state: directory
+    group: nogroup
+    owner: mautrix-facebook
+    mode: u=rwx,g=rx,o=
+
 - name: Copy the daemon configuration
   template:
     src: bridge_facebook.service
diff --git a/roles/matrix-bridge-facebook/templates/config.yaml b/roles/matrix-bridge-facebook/templates/config.yaml
index f24a8cf..8e0e594 100644
--- a/roles/matrix-bridge-facebook/templates/config.yaml
+++ b/roles/matrix-bridge-facebook/templates/config.yaml
@@ -247,7 +247,7 @@ logging:
         file:
             class: logging.handlers.RotatingFileHandler
             formatter: normal
-            filename: ./mautrix-facebook.log
+            filename: /var/log/bridge_facebook/mautrix-facebook.log
             maxBytes: 10485760
             backupCount: 10
         console: