admin still must set deprecated group, as it will be overwritten by a…

… following migration
JustaPenguin · Feb 18, 2020 · a2cfb0f · a2cfb0f
1 parent 9ef82f8
commit a2cfb0f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,13 @@
+v1.7.3-1
+--------
+
+Fixes:
+
+* Fixes an issue where new multiserver setups of Server Manager would go into a redirect loop when creating new servers.
+  If you experienced this issue, please try setting the multiserver up from scratch (i.e. delete the shared_store.json and servers folders).
+
+---
+
 v1.7.3
 ------
 

diff --git a/migrations.go b/migrations.go
@@ -132,6 +132,8 @@ func addAdminAccount(rs Store) error {
 	account := NewAccount()
 	account.Name = adminUserName
 	account.DefaultPassword = "servermanager"
+	// DeprecatedGroup must be assigned to here, since a following migration will just re-set-up Groups.
+	account.DeprecatedGroup = GroupAdmin
 	account.Groups[serverID] = GroupAdmin
 
 	return rs.UpsertAccount(account)