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.

Sign up for GitHub

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

Jump to bottom

Element-R: scary warnings from key gossip store #26527

Closed
richvdh opened this issue Nov 8, 2023 · 2 comments
Closed

Element-R: scary warnings from key gossip store #26527

richvdh opened this issue Nov 8, 2023 · 2 comments
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust S-Tolerable Low/no impact on users Team: Crypto Z-Awaiting-Element-R Issues in code which will be replaced by the port of Element's crypto layer to Rust Z-Labs

Comments

@richvdh
Copy link
Member

richvdh commented Nov 8, 2023 

2023-11-07T23:14:43.637Z W Error decrypting event (id=$tujhmaqviaS12i4L6Ces2Ig80H27c8s51FvPAOKZPM8 type=m.room.encrypted sender=@matthew:matrix.org room=!UcgyhoigetVICUfvRw:matrix.org ts=2023-11-07T21:51:05.603Z): DecryptionError[msg: DomException ConstraintError (0): Unable to add key to index 'by_info': at least one key does not satisfy the uniqueness requirements., session: NRTGMk0PiweE2HYUKH2d0o4+cbMnH52lo5l6iRdcpBc|s62/7/yixJuGRafU4+g7OXyK2JaQBmLWz7gH3+ecZU4]

I think this only happens when (a) decryption failed and (b) there is already a keyshare request in flight; so it is harmless, other than obscuring the underlying reason for decryption. Still it is a source of logspam and we should fix it.
@richvdh richvdh added the A-Element-R Issues affecting the port of Element's crypto layer to Rust label Nov 8, 2023
@github-actions github-actions bot added the Z-Labs label Nov 8, 2023
@dbkr dbkr added S-Tolerable Low/no impact on users Team: Crypto labels Nov 8, 2023
@richvdh richvdh added the Z-Awaiting-Element-R Issues in code which will be replaced by the port of Element's crypto layer to Rust label Nov 9, 2023
@richvdh richvdh mentioned this issue Dec 2, 2023
Closed
@richvdh
Copy link
Member Author

richvdh commented Dec 4, 2023

These will go away as a side-effect of #26524

@richvdh
Copy link
Member Author

richvdh commented Dec 13, 2023

Closing for now, since key-requests are disabled.

@richvdh richvdh closed this as completed Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust S-Tolerable Low/no impact on users Team: Crypto Z-Awaiting-Element-R Issues in code which will be replaced by the port of Element's crypto layer to Rust Z-Labs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dbkr @richvdh