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

Enabling encryption produces a spurious verification prompt #27305

Closed
ara4n opened this issue Apr 9, 2024 · 2 comments
Closed

Enabling encryption produces a spurious verification prompt #27305

ara4n opened this issue Apr 9, 2024 · 2 comments
Labels
A-E2EE O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Apr 9, 2024

Steps to reproduce

  1. Take a 4 person private room (unencrypted), flagged as a DM
  2. Enable encryption on it
  3. See:

Screenshot 2024-04-09 at 09 43 33

...except:

a) i've already verified Amandine
b) it's a 4-person room; why pick on Amandine specifically?

Outcome

What did you expect?

If it's going to prompt me to verify the members of a room, please prompt me to verify the unverified ones, not the verified ones.

Operating system

No response

Application version

Element Nightly version: 2024040801 Crypto version: Rust SDK 0.7.0 (61b175b), Vodozemac 0.5.1

How did you install the app?

No response

Homeserver

No response

Will you send logs?

No
@ara4n ara4n added the T-Defect label Apr 9, 2024
@t3chguy
Copy link
Member

t3chguy commented Apr 9, 2024 •
edited
Loading

flagged as a DM

Because of this, DMs are a 1:1 relationship still until Canonical DMs, this room is considered a DM with Amandine.

@MidhunSureshR MidhunSureshR added S-Minor Impairs non-critical functionality or suitable workarounds exist A-E2EE O-Occasional Affects or can be seen by some users regularly or most users rarely labels Apr 30, 2024
@richvdh
Copy link
Member

richvdh commented Jan 21, 2025

Seems like basically a duplicate of #15831

@richvdh richvdh closed this as completed Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-E2EE O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ara4n @richvdh @t3chguy @MidhunSureshR