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: sending messages takes several seconds #26375

Closed
richvdh opened this issue Oct 16, 2023 · 9 comments · Fixed by matrix-org/matrix-js-sdk#3909
Closed

Element-R: sending messages takes several seconds #26375

richvdh opened this issue Oct 16, 2023 · 9 comments · Fixed by matrix-org/matrix-js-sdk#3909
Assignees
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust T-Defect

Comments

@richvdh
Copy link
Member

richvdh commented Oct 16, 2023

Every time you send a message, we make a /keys/claim request for each device where we don't already have an olm session. This can take several seconds.

If there are devices in the room which have run out of OTKs, we will do this for every single message.

@richvdh richvdh added T-Defect A-Element-R Issues affecting the port of Element's crypto layer to Rust labels Oct 16, 2023
@richvdh
Copy link
Member Author

richvdh commented Oct 16, 2023

matrix-org/matrix-rust-sdk#281 will address this.

@ara4n
Copy link
Member

ara4n commented Oct 16, 2023

moved comment to element-hq/element-meta#2149 at @richvdh's request

@richvdh
Copy link
Member Author

richvdh commented Oct 19, 2023

might be improved by #26382; to revisit once that is merged

@richvdh richvdh self-assigned this Oct 25, 2023
@richvdh
Copy link
Member Author

richvdh commented Oct 25, 2023

might be improved by #26382; to revisit once that is merged

It's still taking me 8 seconds to send each message in #megolm:matrix.org, of which 5 seconds is spent waiting for a /keys/claim request.

@richvdh
Copy link
Member Author

richvdh commented Oct 30, 2023

/keys/claim being slow (and repeated) is not the only problem here; see also matrix-org/matrix-rust-sdk#2793

@richvdh
Copy link
Member Author

richvdh commented Nov 1, 2023

Also: matrix-org/matrix-rust-sdk#2804

@richvdh
Copy link
Member Author

richvdh commented Nov 8, 2023

We need to improve the logging in the message send path. It's generally impossible to tell what is going on and how long it takes.

@richvdh
Copy link
Member Author

richvdh commented Nov 21, 2023

@richvdh
Copy link
Member Author

richvdh commented Nov 25, 2023

Currently blocked on matrix-org/matrix-rust-sdk-crypto-wasm#58

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 T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants