This repository was archived by the owner on Apr 26, 2024. It is now read-only.
This repository was archived by the owner on Apr 26, 2024. It is now read-only.
/sync is not bundling aggregations for limited rooms #11476
Description
MSC2675 says that /sync
is one of the APIs which should bundle aggregations:
GET /sync
, only for room sections in the response wherelimited
field istrue
; this amounts to all rooms in the response if thesince
request parameter was not passed, also known as an initial sync.
Currently it always disables bundled aggregations for this endpoint, see #5220.
We should also double check the other endpoints:
-
GET /rooms/{roomId}/messages
-- see code -
GET /rooms/{roomId}/context
-- see code (note that this doesn't bundle them for state events, unclear if this is "fine" or not) -
GET /rooms/{roomId}/event/{eventId}
-- see code -
GET /sync
-
GET /relations
-- see Include bundled events when requesting relations #11087