diff options
| author | Neil Alexander <neilalexander@users.noreply.github.com> | 2022-11-02 09:34:19 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-11-02 09:34:19 +0000 |
| commit | 3db9e98456b3580f230035c186dc4216f2043908 (patch) | |
| tree | 594ee26ba39531d1428e09a955d338a5183a3d29 /syncapi/storage/sqlite3/current_room_state_table.go | |
| parent | 8a1904ffe593b888954ba85a42fd869095163d27 (diff) | |
Don't limit `"state"` (#2849)
This is apparently some incorrect behaviour that we built as a result of
a spec bug (matrix-org/matrix-spec#1314) where we were applying a filter
to the `"state"` section of the `/sync` response incorrectly. The client
then has no way to know that the state was limited.
This PR removes the state limiting, which probably also helps #2842.
Diffstat (limited to 'syncapi/storage/sqlite3/current_room_state_table.go')
| -rw-r--r-- | syncapi/storage/sqlite3/current_room_state_table.go | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/syncapi/storage/sqlite3/current_room_state_table.go b/syncapi/storage/sqlite3/current_room_state_table.go index ff45e786..7a381f68 100644 --- a/syncapi/storage/sqlite3/current_room_state_table.go +++ b/syncapi/storage/sqlite3/current_room_state_table.go @@ -277,7 +277,8 @@ func (s *currentRoomStateStatements) SelectCurrentState( }, stateFilter.Senders, stateFilter.NotSenders, stateFilter.Types, stateFilter.NotTypes, - excludeEventIDs, stateFilter.ContainsURL, stateFilter.Limit, FilterOrderNone, + excludeEventIDs, stateFilter.ContainsURL, 0, + FilterOrderNone, ) if err != nil { return nil, fmt.Errorf("s.prepareWithFilters: %w", err) |