refactor: Make ClientBuilder safer to use when restoring a session#5874
Conversation
This may be dangerous when done while restoring an existing session.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5874 +/- ##
==========================================
+ Coverage 88.59% 88.61% +0.01%
==========================================
Files 363 363
Lines 102888 102888
Branches 102888 102888
==========================================
+ Hits 91153 91171 +18
+ Misses 7496 7475 -21
- Partials 4239 4242 +3 ☔ View full report in Codecov by Sentry. |
CodSpeed Performance ReportMerging #5874 will not alter performanceComparing Summary
|
|
Looks like we need a complement-crypto update to go with this? https://github.com/matrix-org/matrix-rust-sdk/actions/runs/19459574501/job/55680392867?pr=5874
|
Yes, I didn't realise this would affect complement-crypto. |
|
Added CC PR here and retrying the bindings test. |
|
@andybalaam it's working now! |
Changes:
Add some docs to discourage using the
ClientBuildermethods that modify the server or homeserver URLs when restoring an existing session.Removes the unused
ffi::Session::homeserver_urlfield, since it can lead to clients believing that modifying that value may have an effect in the SDK behaviour.Public API changes documented in changelogs (optional)
Signed-off-by: