From c626431c4145f12d05e4c77919a78f4a20c7fe2a Mon Sep 17 00:00:00 2001
From: Catalan Lover <48515417+FSG-Cat@users.noreply.github.com>
Date: Wed, 24 May 2023 22:58:42 +0200
Subject: [PATCH] Revert #2698 forget_rooms_on_leave to upstream value

Reverts forget_rooms_on_leave to upstream as this is too dangerous to change in a breaking way like this is doing. Especially doing it together with a Call to Action to update your Synapses causing this change to enter all playbook users playbooks who update.

Its safe to keep the version bump part of #2698 but not safe to keep this change that this commit is undoing by setting the value to false. This means our config now includes the value like a upstream config would but we keep it at false just like upstream does.
---
 .../custom/matrix-synapse/templates/synapse/homeserver.yaml.j2  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/roles/custom/matrix-synapse/templates/synapse/homeserver.yaml.j2 b/roles/custom/matrix-synapse/templates/synapse/homeserver.yaml.j2
index ff2533977..e8787aa8a 100644
--- a/roles/custom/matrix-synapse/templates/synapse/homeserver.yaml.j2
+++ b/roles/custom/matrix-synapse/templates/synapse/homeserver.yaml.j2
@@ -143,7 +143,7 @@ default_room_version: {{ matrix_synapse_default_room_version|to_json }}
 
 # Set to true to automatically forget rooms for users when they leave them, either
 # normally or via a kick or ban. Defaults to false.
-forget_rooms_on_leave: true
+forget_rooms_on_leave: false
 
 # The GC threshold parameters to pass to `gc.set_threshold`, if defined
 #