Add comment as to why we return the entire content for ACLs

S7evinK · S7evinK · commit 3125f4b03377 · 2024-02-21T13:21:05.000+01:00
diff --git a/roomserver/storage/tables/interface.go b/roomserver/storage/tables/interface.go
@@ -236,6 +236,8 @@ func ExtractContentValue(ev *types.HeaderedEvent) string {
 	case "m.room.guest_access":
 		key = "guest_access"
 	case "m.room.server_acl":
+		// We need the entire content and not only one key, so we can use it
+		// on startup to generate the ACLs. This is merely a workaround.
 		return string(content)
 	}
 	result := gjson.GetBytes(content, key)

Original file line number	Diff line number	Diff line change
`@@ -236,6 +236,8 @@ func ExtractContentValue(ev *types.HeaderedEvent) string {`
`236`	`236`	`case "m.room.guest_access":`
`237`	`237`	`key = "guest_access"`
`238`	`238`	`case "m.room.server_acl":`
	`239`	`+ // We need the entire content and not only one key, so we can use it`
	`240`	`+ // on startup to generate the ACLs. This is merely a workaround.`
`239`	`241`	`return string(content)`
`240`	`242`	`}`
`241`	`243`	`result := gjson.GetBytes(content, key)`