-
Notifications
You must be signed in to change notification settings - Fork 241
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(cargo): Update ruma
#4039
Conversation
8ea2fa7
to
2f807ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What the fuck are you doing on a Friday here. 😾
Haha, eh, just a little something while getting bored. |
dfd8d46
to
95ea8e8
Compare
This patch changes the behaviour of `timeline_limit` in sliding sync requests. It previously was sticky, but since it's now mandatory with MSC4186, it's preferable it to be non-sticky, otherwise in some scenarios it might default to 0 (its default value). How? If the server doesn't reply with our `txn_id` (because it doesn't support sticky parameters or because it misses a `txn_id`), the next request will be built with a default `timeline_limit` value, which is zero, and won't get updated to the `timeline_limit` value from `SlidingSyncListStickyParameters`. This is not good. Instead, we must consider `timeline_limit` as non-sticky, and moves it from `SlidingSyncListStickyParameters` to `SlidingSyncListInner`. This is what this patch does.
95ea8e8
to
9807357
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4039 +/- ##
==========================================
+ Coverage 84.66% 84.67% +0.01%
==========================================
Files 269 269
Lines 28715 28711 -4
==========================================
Hits 24311 24311
+ Misses 4404 4400 -4 ☔ View full report in Codecov by Sentry. |
This patch updates
ruma
to include:sync::v5::Response::initial
ruma/ruma#1913sync::v5::Request::timeline_limit
is not optional ruma/ruma#1914