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
Validate backup private key before migrating it #4114
Validate backup private key before migrating it #4114
Conversation
Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
…s://github.com/matrix-org/matrix-js-sdk into valere/element-r/migrate_local_trust_if_needed
Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
8cb1334
to
bf3e6d6
Compare
cda5200
to
6852682
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.
Looks good, a couple of minor points. I hope other reviewers will be able to comment on whether the expected responses from the server are correct - I don't know.
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.
Generally looks sensible to me but a few comments
Fixes element-hq/element-web#27324
Add some checks to validate that the legacy cached private key matches the current backup when migrating (it could have changed while the session was closed).
At the moment we need to have libolm available to check the key validity, it's was needed before that. Maybe see if we could expose some rust API?
Checklist
public
/exported
symbols have accurate TSDoc documentation.