Removing unnecesary federation configs (#14674)

RocketChat · May 30, 2019 · c178ddb · c178ddb
1 parent 484ee10
commit c178ddb
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/server/startup/migrations/index.js b/server/startup/migrations/index.js
@@ -143,4 +143,6 @@ import './v142';
 import './v143';
 import './v144';
 import './v145';
+import './v146';
+import './v147';
 import './xrun';
diff --git a/server/startup/migrations/v147.js b/server/startup/migrations/v147.js
@@ -0,0 +1,22 @@
+import { Migrations } from '../../../app/migrations/server';
+import { Settings } from '../../../app/models';
+
+const validSettings = [
+	'FEDERATION_Discovery_Method',
+	'FEDERATION_Domain',
+	'FEDERATION_Enabled',
+	'FEDERATION_Hub_URL',
+	'FEDERATION_Public_Key',
+	'FEDERATION_Status',
+	'FEDERATION_Test_Setup',
+];
+
+Migrations.add({
+	version: 147,
+	up() {
+		Settings.remove({ $and: [{ _id: /FEDERATION/ }, { _id: { $nin: validSettings } }] });
+	},
+	down() {
+		// Down migration does not apply in this case
+	},
+});