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

Element-R: creates new cross-signing identity if initial /keys/query takes longer than 1s #25773

Closed
richvdh opened this issue Jul 12, 2023 · 1 comment · Fixed by matrix-org/matrix-js-sdk#3599
Assignees
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@richvdh
Copy link
Member

richvdh commented Jul 12, 2023

Sometimes, when you log in, Element-R skips straight through to the home page rather than letting you verify your new device with the cross-signing keys.

I think the problem is related to the fact we only wait for a second for /keys/query requests to take place... and we don't even start making them until the first /sync completes.

@richvdh richvdh added T-Defect S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Occasional Affects or can be seen by some users regularly or most users rarely A-Element-R Issues affecting the port of Element's crypto layer to Rust labels Jul 12, 2023
@richvdh richvdh self-assigned this Jul 12, 2023
@richvdh
Copy link
Member Author

richvdh commented Jul 12, 2023

What happens here is:

So there are two problems here; one is the incorrect result of userHasCrossSigningKeys which causes us to call bootstrapCrossSigning in the first place, and the other is that bootstrapCrossSigning does not check secret storage (now raised as #25781).

@richvdh richvdh changed the title Element-R: creates new cross-signing identity if initial /sync takes longer than 1s Element-R: creates new cross-signing identity if initial /keys/query takes longer than 1s Jul 12, 2023
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-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant