Skip to content
Snippets Groups Projects
Unverified Commit f58b300d authored by Patrick Cloke's avatar Patrick Cloke Committed by GitHub
Browse files

Do not attempt to bundled aggregations for /members and /state. (#11623)

Both of those APIs return state events, which will not have bundled
aggregations added anyway.
parent 15bb1c85
No related branches found
No related tags found
No related merge requests found
Fix a long-standing bug where responses included bundled aggregations when they should not, per [MSC2675](https://github.com/matrix-org/matrix-doc/pull/2675).
...@@ -246,9 +246,7 @@ class MessageHandler: ...@@ -246,9 +246,7 @@ class MessageHandler:
room_state = room_state_events[membership_event_id] room_state = room_state_events[membership_event_id]
now = self.clock.time_msec() now = self.clock.time_msec()
events = await self._event_serializer.serialize_events( events = await self._event_serializer.serialize_events(room_state.values(), now)
room_state.values(), now, bundle_aggregations=True
)
return events return events
async def get_joined_members(self, requester: Requester, room_id: str) -> dict: async def get_joined_members(self, requester: Requester, room_id: str) -> dict:
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment