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: stuck in a loop calling /keys/query #26194

Closed
richvdh opened this issue Sep 18, 2023 · 2 comments · Fixed by matrix-org/matrix-js-sdk#3757
Closed

Element-R: stuck in a loop calling /keys/query #26194

richvdh opened this issue Sep 18, 2023 · 2 comments · Fixed by matrix-org/matrix-js-sdk#3757
Assignees
@BillCarsonFr
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust T-Defect Team: Crypto

Comments

@richvdh
Copy link
Member

richvdh commented Sep 18, 2023 •
edited
Loading

It seems to be continually calling /keys/query for itself
@richvdh richvdh added T-Defect Team: Crypto A-Element-R Issues affecting the port of Element's crypto layer to Rust labels Sep 18, 2023
@BillCarsonFr
Copy link
Member

So the loop is due to the userHasCrossSigningKeys() that is called as part of DeviceListener#doRecheck().

The userHasCrossSigningKeys() is forcing a keys/query that will emit a CryptoEvent.UserTrustStatusChanged,
and DeviceListener will trigger a doRecheck on that event

@BillCarsonFr BillCarsonFr mentioned this issue Sep 26, 2023
Merged
1 task
@richvdh
Copy link
Member Author

richvdh commented Sep 28, 2023

I think we are still using the old version of the rust library so this is not yet fixed

@richvdh richvdh reopened this Sep 28, 2023
@richvdh richvdh mentioned this issue Sep 28, 2023
Merged
@t3chguy t3chguy mentioned this issue Mar 12, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BillCarsonFr BillCarsonFr

Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust T-Defect Team: Crypto
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bump matrix-sdk-crypto-wasm to 1.3.0 matrix-org/matrix-js-sdk
2 participants
@richvdh @BillCarsonFr