Skip to content
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

Improve fallback key behaviour #2037

Merged
merged 8 commits into from
Dec 10, 2021

Conversation

uhoreg
Copy link
Member

@uhoreg uhoreg commented Nov 23, 2021

  • tracks whether the fallback key has already been published
  • clear out old fallback key some time after it's been replaced

Note: do not merge until libolm 3.2.7 is released. (Also, needs a change to package.json to use the newer libolm) (test failures are due to this) Done


Here's what your changelog entry will look like:

✨ Features

  • Improve fallback key behaviour (#2037).

@uhoreg uhoreg requested a review from a team as a code owner November 23, 2021 22:29
@uhoreg uhoreg marked this pull request as draft November 23, 2021 22:30
@uhoreg uhoreg changed the title Fix fallback key behaviour Improve fallback key behaviour Nov 23, 2021
@turt2live turt2live removed the request for review from a team November 23, 2021 23:22
@uhoreg uhoreg requested a review from a team December 8, 2021 18:55
@uhoreg uhoreg marked this pull request as ready for review December 8, 2021 18:55
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yay, improved fallback keys

src/crypto/index.ts Outdated Show resolved Hide resolved
Co-authored-by: Travis Ralston <travpc@gmail.com>
@uhoreg
Copy link
Member Author

uhoreg commented Dec 9, 2021

Apparently, ae68f00 makes jest run for over an hour, which is probably not ideal. Will try to figure out a workaround.

@t3chguy
Copy link
Member

t3chguy commented Dec 9, 2021

Look into jest.useFakeTimers()

@uhoreg
Copy link
Member Author

uhoreg commented Dec 9, 2021

I think my changes exceed the "needs re-review" threshold. (I also added support for unprefixed properties, to match matrix-org/synapse#11541)

@uhoreg uhoreg requested a review from turt2live December 9, 2021 19:00
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks :)

@uhoreg uhoreg merged commit dd8c157 into matrix-org:develop Dec 10, 2021
su-ex added a commit to SchildiChat/matrix-js-sdk that referenced this pull request Dec 21, 2021
* Improve fallback key behaviour ([\matrix-org#2037](matrix-org#2037)).
* Add new room event filter fields ([\matrix-org#2051](matrix-org#2051)).
* Add method to fetch /account/whoami ([\matrix-org#2046](matrix-org#2046)).
* Filter out falsey opts in /relations API hits ([\matrix-org#2059](matrix-org#2059)). Fixes element-hq/element-web#20137.
* Fix paginateEventTimeline resolve to boolean ([\matrix-org#2054](matrix-org#2054)).
* Fix incorrect MSC3089 typings and add null checks ([\matrix-org#2049](matrix-org#2049)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants