Skip to content

make sure setDefaultKeyId resolves even if no change is made - #5485

Merged
andybalaam merged 1 commit into
matrix-org:developfrom
uhoreg:setdefaultkeyid_fix_noop
Aug 17, 2026
Merged

make sure setDefaultKeyId resolves even if no change is made#5485
andybalaam merged 1 commit into
matrix-org:developfrom
uhoreg:setdefaultkeyid_fix_noop

Conversation

@uhoreg

@uhoreg uhoreg commented Aug 14, 2026

Copy link
Copy Markdown
Member

fixes #5474

Make setDefaultKeyId just setAccountData and return, rather than listening for the value to come back down the sync. setAccountData already does that now, when it actually tries to change the value, and does not make an HTTP call if we try to set the account data value to the current value, which means that setDefaultKeyId would never get notified of the value being changed (since it doesn't get changed), and so would never return.

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • Sign-off given on the changes (see CONTRIBUTING.md).

@uhoreg
uhoreg force-pushed the setdefaultkeyid_fix_noop branch from e1f8da3 to 19c5dbb Compare August 14, 2026 21:52
@andybalaam
andybalaam added this pull request to the merge queue Aug 17, 2026
Merged via the queue into matrix-org:develop with commit b4b8d5e Aug 17, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RustCrypto.resetEncryption() never resolves when secret storage account data is already empty

3 participants