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

Stuck messages in the timeline #17966

Closed
ara4n opened this issue Jul 12, 2021 · 13 comments
Closed

Stuck messages in the timeline #17966

ara4n opened this issue Jul 12, 2021 · 13 comments
Labels
A-Timeline 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 X-Needs-Info This issue is blocked awaiting information from the reporter Z-Backend Z-Synapse

Comments

@ara4n
Copy link
Member

ara4n commented Jul 12, 2021

Over the last few weeks i've had ~4 instances where a sent message gets stuck at the bottom of the timeline because we didn't successfully receive an echo down /sync. I rageshaked; @richvdh took a look with jaeger and says there may be two competing /sync requests which tried to receive the message, and so it may have fallen between the cracks. As far as I know, eleweb should never be running overlapping sync loops - so it sounds like something is going worryingly wrong here.

@ara4n ara4n added the T-Defect label Jul 12, 2021
@ara4n
Copy link
Member Author

ara4n commented Jul 12, 2021

this is also matrix-org/synapse#9424 (comment)

@turt2live
Copy link
Member

I'm slightly worried this is a synapse-side threading issue btw. Mjolnir and such are seeing duplicate events down /sync, which sounds like some sort of corrupted aggregated state.

@t3chguy
Copy link
Member

t3chguy commented Jul 12, 2021

Duplicate of #17471

@t3chguy t3chguy marked this as a duplicate of #17471 Jul 12, 2021
@t3chguy t3chguy closed this as completed Jul 12, 2021
@ara4n
Copy link
Member Author

ara4n commented Jul 12, 2021

I don't think this is a precise dup of the #17471, given @richvdh's investigation with jaeger showed two possible /sync reqs from eleweb which would have caught the event. Whereas #17471 says...

This is a bug when the server never remote-echoes the event back down /sync

which is not the case here.

@ara4n ara4n reopened this Jul 12, 2021
@richvdh
Copy link
Member

richvdh commented Jul 13, 2021

My money is currently on synapse not returning the event in either sync request, to be clear.

@dbkr dbkr added A-Timeline 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 labels Aug 19, 2021
@dbkr
Copy link
Member

dbkr commented Aug 19, 2021

I just got this again and it's worse now because the new 'sent' tick is just on the latest message, so i you send another message afterwards, the latter one can get through but then the two messages just look like they've been re-ordered, and the 'sent' tick is on the one that actually didn't send at all!

Screenshot 2021-08-19 at 11 08 24

@nrdxp
Copy link

nrdxp commented Aug 21, 2021

Is anyone aware of a workaround to resolve this. It's quite annoying 😓

@JokerGermany
Copy link

JokerGermany commented Aug 24, 2021

@novocaine
Copy link
Contributor

novocaine commented Nov 9, 2021

@ara4n thinks this is an E2EE client bug .. need to understand if that's tracked elsewhere

@novocaine novocaine added the X-Needs-Info This issue is blocked awaiting information from the reporter label Nov 9, 2021
@Palid
Copy link
Contributor

Palid commented Nov 12, 2021

This issue also exists in the Android version of the app. @kittykat are you aware of it?

@Palid
Copy link
Contributor

Palid commented Nov 12, 2021

If I remember correctly it usually happened whenever I had some backlog to sync and I sent the message and closed the app/changed room focus (either or) before backlog did properly sync, which left me with this hanging message.

@BillCarsonFr
Copy link
Member

New occurence on web for me https://github.com/matrix-org/element-web-rageshakes/issues/10049

@t3chguy
Copy link
Member

t3chguy commented May 11, 2022

Closing in favour of more generic element-hq/element-meta#1597

@t3chguy t3chguy closed this as completed May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Timeline 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 X-Needs-Info This issue is blocked awaiting information from the reporter Z-Backend Z-Synapse
Projects
None yet
Development

No branches or pull requests