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

Message sending retry management #250

Closed
stefanceriu opened this issue Oct 25, 2022 · 3 comments
Closed

Message sending retry management #250

stefanceriu opened this issue Oct 25, 2022 · 3 comments
Assignees
@pixlwave
Labels
T-User Story Team: Element X Platform X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue Z-BBQ-Beta Required for ElementX - BBQ - Beta

Comments

@stefanceriu
Copy link
Member

stefanceriu commented Oct 25, 2022 •
edited
Loading

Build mecanism for displaying and retrying messages that failed to send.
@ara4n
Copy link
Member

ara4n commented Jan 10, 2023

n.b. that the retrying has to be queued in order (in the rust-sdk, presumably). given the send API doesn't let you define an ordering relative to other sends (matrix-org/matrix-spec#1316), we have no choice but serialise each /send and wait for the previous one to complete with a 200 OK before sending the next one.

@stefanceriu stefanceriu added the X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue label Jan 10, 2023
@pixlwave
Copy link
Member

@ara4n Am I right in thinking its safe to queue send and retry per room as opposed to globally?

@pixlwave pixlwave self-assigned this Feb 23, 2023
@VolkerJunginger
Copy link

Duplicate to https://github.com/vector-im/internal-planning/issues/312

@VolkerJunginger VolkerJunginger closed this as not planned Won't fix, can't repro, duplicate, stale Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pixlwave pixlwave

Labels
T-User Story Team: Element X Platform X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue Z-BBQ-Beta Required for ElementX - BBQ - Beta
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kittykat @stefanceriu @ara4n @pixlwave @VolkerJunginger