From f39f134e12c9100c8b1375f333749743d254c584 Mon Sep 17 00:00:00 2001
From: Patrick Cloke <patrickc@matrix.org>
Date: Thu, 31 Mar 2022 14:22:36 -0400
Subject: [PATCH] Disable groups by default.

---
 changelog.d/12344.removal      | 1 +
 docs/upgrade.md                | 7 +++++++
 synapse/config/experimental.py | 2 +-
 tests/rest/admin/test_admin.py | 1 +
 4 files changed, 10 insertions(+), 1 deletion(-)
 create mode 100644 changelog.d/12344.removal

diff --git a/changelog.d/12344.removal b/changelog.d/12344.removal
new file mode 100644
index 000000000000..ecefa76d8ea5
--- /dev/null
+++ b/changelog.d/12344.removal
@@ -0,0 +1 @@
+The groups/communities feature in Synapse has been disabled by default.
diff --git a/docs/upgrade.md b/docs/upgrade.md
index f0c0a3998d98..3d518c81c3b0 100644
--- a/docs/upgrade.md
+++ b/docs/upgrade.md
@@ -85,6 +85,13 @@ process, for example:
     dpkg -i matrix-synapse-py3_1.3.0+stretch1_amd64.deb
     ```
 
+# Upgrading to v1.58.0
+
+## Groups/communities feature has been disabled by default
+
+The non-standard groups/communities feature in Synapse has been disabled by default
+and will be removed in Synapse v1.61.0.
+
 # Upgrading to v1.57.0
 
 ## Changes to database schema for application services
diff --git a/synapse/config/experimental.py b/synapse/config/experimental.py
index 447476fbfac5..6077794aad8e 100644
--- a/synapse/config/experimental.py
+++ b/synapse/config/experimental.py
@@ -77,7 +77,7 @@ def read_config(self, config: JsonDict, **kwargs: Any) -> None:
         self.msc3720_enabled: bool = experimental.get("msc3720_enabled", False)
 
         # The deprecated groups feature.
-        self.groups_enabled: bool = experimental.get("groups_enabled", True)
+        self.groups_enabled: bool = experimental.get("groups_enabled", False)
 
         # MSC2654: Unread counts
         self.msc2654_enabled: bool = experimental.get("msc2654_enabled", False)
diff --git a/tests/rest/admin/test_admin.py b/tests/rest/admin/test_admin.py
index 849d00ab4d94..40571b753a9a 100644
--- a/tests/rest/admin/test_admin.py
+++ b/tests/rest/admin/test_admin.py
@@ -63,6 +63,7 @@ def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
         self.other_user = self.register_user("user", "pass")
         self.other_user_token = self.login("user", "pass")
 
+    @unittest.override_config({"experimental_features": {"groups_enabled": True}})
     def test_delete_group(self) -> None:
         # Create a new group
         channel = self.make_request(