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

Retry processing potential poll events after decryption #3246

Merged
merged 5 commits into from
Apr 6, 2023

Conversation

weeman1337
Copy link
Contributor

@weeman1337 weeman1337 commented Apr 3, 2023

The issue here was that events in failed decryption state were not processed again after another decryption event.

fixes element-hq/element-web#24568

PSB-300

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

Here's what your changelog entry will look like:

🐛 Bug Fixes

Copy link
Contributor

@kerryarchibald kerryarchibald left a comment

Choose a reason for hiding this comment

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

LGTM

src/models/room.ts Show resolved Hide resolved
richvdh and others added 4 commits April 4, 2023 06:35
This gets rid of the rather pointless "default" module in the generated docs.
* Move SecretStorageKeyInfo interfaces out to a new module

* Replace usages of ISecretStorageKeyInfo with SecretStorageKeyDescription
@weeman1337 weeman1337 added this pull request to the merge queue Apr 6, 2023
Merged via the queue into develop with commit d7e165a Apr 6, 2023
@weeman1337 weeman1337 deleted the weeman1337/retry-poll-events branch April 6, 2023 11:17
su-ex added a commit to SchildiChat/matrix-js-sdk that referenced this pull request Apr 25, 2023
* Change `Store.save()` to return a `Promise` ([\matrix-org#3221](matrix-org#3221)). Contributed by @texuf.
* Add typedoc-plugin-mdn-links ([\matrix-org#3292](matrix-org#3292)).
* Annotate events with executed push rule ([\matrix-org#3284](matrix-org#3284)). Contributed by @kerryarchibald.
* Element-R: pass device list change notifications into rust crypto-sdk ([\matrix-org#3254](matrix-org#3254)). Fixes element-hq/element-web#24795. Contributed by @florianduros.
* Support for MSC3882 revision 1 ([\matrix-org#3228](matrix-org#3228)). Contributed by @hughns.
* Fix screen sharing on Firefox 113 ([\matrix-org#3282](matrix-org#3282)). Contributed by @tulir.
* Retry processing potential poll events after decryption ([\matrix-org#3246](matrix-org#3246)). Fixes element-hq/element-web#24568.
* Element-R: handle events which arrive before their keys ([\matrix-org#3230](matrix-org#3230)). Fixes element-hq/element-web#24489.
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.

Poll not visible
3 participants