diff options
| author | Patrick Cloke <clokep@users.noreply.github.com> | 2021-12-20 14:14:38 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-12-20 14:14:38 -0500 |
| commit | dd4778875213d9cb8be7ee71d32751fbd6cdaba2 (patch) | |
| tree | bc638f5df3a1e60a5cb2b15b8177810c2f49d700 /synapse/rest/admin | |
| parent | Improve opentracing support for `ResponseCache` (#11607) (diff) | |
| download | synapse-dd4778875213d9cb8be7ee71d32751fbd6cdaba2.tar.xz | |
Do not bundle aggregations for APIs which shouldn't include them. (#11592)
And make bundling aggregations opt-in, instead of opt-out to avoid having APIs to include extraneous data (and being much heavier than necessary).
Diffstat (limited to 'synapse/rest/admin')
| -rw-r--r-- | synapse/rest/admin/rooms.py | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/synapse/rest/admin/rooms.py b/synapse/rest/admin/rooms.py index 17c6df1cc8..6030373ebc 100644 --- a/synapse/rest/admin/rooms.py +++ b/synapse/rest/admin/rooms.py @@ -745,13 +745,19 @@ class RoomEventContextServlet(RestServlet): time_now = self.clock.time_msec() results["events_before"] = await self._event_serializer.serialize_events( - results["events_before"], time_now + results["events_before"], + time_now, + bundle_aggregations=True, ) results["event"] = await self._event_serializer.serialize_event( - results["event"], time_now + results["event"], + time_now, + bundle_aggregations=True, ) results["events_after"] = await self._event_serializer.serialize_events( - results["events_after"], time_now + results["events_after"], + time_now, + bundle_aggregations=True, ) results["state"] = await self._event_serializer.serialize_events( results["state"], time_now |