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

Session manager shows incorrect verification status #26989

Closed
uhoreg opened this issue Feb 8, 2024 · 2 comments
Closed

Session manager shows incorrect verification status #26989

uhoreg opened this issue Feb 8, 2024 · 2 comments
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Team: Crypto Z-Labs

Comments

@uhoreg
Copy link
Member

uhoreg commented Feb 8, 2024

Steps to reproduce

Not sure how to reproduce, but could be related to rust migration, as I'm pretty sure it was displaying correctly before migrating.

Outcome

What did you expect?

Session manager should show all my devices as verified

What happened instead?

Session manager shows all my devices (including itself) as unverified:

image

User info panel in sidebar shows all my devices as verified:

image

Session manager on another device (develop.element.io, was always running rust crypto) shows everything correctly

image

Operating system

Debian Linux

Application version

Element nightly 2024020701, Rust SDK 0.7.0 (691ec63), Vodozemac 0.5.0

How did you install the app?

element.io Debian package repository

Homeserver

uhoreg.ca, Synapse 1.100.0

Will you send logs?

Yes

@uhoreg uhoreg added the T-Defect label Feb 8, 2024
@uhoreg
Copy link
Member Author

uhoreg commented Feb 8, 2024

Looks like the cause was that the client was missing some cross-signing key, and had to fetch it from SSSS. So probably not a critical bug, but it's not nice that the session manager shows different results from the user info panel, and that it doesn't indicate what the problem is.

@florianduros florianduros added S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Uncommon Most users are unlikely to come across this or unexpected workflow A-Element-R Issues affecting the port of Element's crypto layer to Rust labels Feb 8, 2024
@github-actions github-actions bot added the Z-Labs label Feb 8, 2024
@github-actions github-actions bot added the Z-Labs label Feb 8, 2024
@BillCarsonFr
Copy link
Member

Closing as duplicate of #27079

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Team: Crypto Z-Labs
Projects
None yet
Development

No branches or pull requests

3 participants