Skip to content

Fix logic error in validation of incoming olm-encrypted messages#2394

Open
tusooa wants to merge 2 commits into
matrix-org:mainfrom
tusooa:patch-4
Open

Fix logic error in validation of incoming olm-encrypted messages#2394
tusooa wants to merge 2 commits into
matrix-org:mainfrom
tusooa:patch-4

Conversation

@tusooa

@tusooa tusooa commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Fix #2363

Pull Request Checklist

Preview: https://pr2394--matrix-spec-previews.netlify.app

@tusooa tusooa requested a review from a team as a code owner June 10, 2026 00:23
@tusooa tusooa force-pushed the patch-4 branch 2 times, most recently from c98f241 to 184ff23 Compare June 10, 2026 00:26
@richvdh richvdh self-requested a review June 23, 2026 15:10

@richvdh richvdh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks very much for sorting this out; I think your descriptions are accurate.

I've made some suggestions for minor clarifications, if you don't mind.

Comment thread content/client-server-api/modules/end_to_end_encryption.md Outdated
Comment thread content/client-server-api/modules/end_to_end_encryption.md
Comment thread changelogs/client_server/newsfragments/2394.clarification Outdated
Signed-off-by: tusooa <tusooa@kazv.moe>

Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
@tusooa tusooa requested a review from zecakeh as a code owner June 24, 2026 04:28
@zecakeh zecakeh requested a review from richvdh June 24, 2026 06:31

@richvdh richvdh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Validation of incoming decrypted events" section contains manifest nonsense

2 participants