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

Thread reply missing from thread in the mix of sending my own thread reply #24466

Closed
MadLittleMods opened this issue Feb 7, 2023 · 6 comments · Fixed by matrix-org/matrix-js-sdk#3618
Assignees
Labels
A-Threads A-Timeline O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Defect X-Release-Blocker

Comments

@MadLittleMods
Copy link
Contributor

MadLittleMods commented Feb 7, 2023

Steps to reproduce

  1. Someone else threads off their own message and sends some thread replies
  2. In the mix of them sending more thread replies, you send a thread reply of your own
  3. Notice a missing thread reply from the other person. Probably some race condition between your message being sent and the other thread replies.

(details in rageshake issue https://github.com/matrix-org/element-web-rageshakes/issues/19979)

Also discussed in an internal room.

Outcome

What did you expect?

All thread replies visible especially since it seems like it reached my client.

What happened instead?

There is a missing thread reply from the other person.

I did see an unread message indicator in the room preview. This phenomena is very hard to notice since I only caught the missing message from a notification on my other computer.

Operating system

Windows 10

Browser information

Chrome Version 109.0.5414.120

URL for webapp

develop.element.io

Application version

Element version: 27f1d99-react-39fe72e53ad6-js-8a3d7d5671e5 Olm version: 3.2.12

Homeserver

matrix.org

Will you send logs?

Yes, https://github.com/matrix-org/element-web-rageshakes/issues/19979

@MadLittleMods MadLittleMods added T-Defect S-Critical Prevents work, causes data loss and/or has no workaround O-Occasional Affects or can be seen by some users regularly or most users rarely A-Threads labels Feb 7, 2023
@MadLittleMods MadLittleMods changed the title Thread reply missing from thread Thread reply missing from thread in the mix of sending my own thread reply Feb 7, 2023
@Johennes
Copy link
Contributor

@MadLittleMods how often have you experienced this since filing the issue?

@MadLittleMods
Copy link
Contributor Author

@Johennes I think 3 times. It seems pretty easy to reproduce in a situation as described above. Was pretty common with two of us going back and forth in a thread (answering someone's Gitter questions).

@Johennes
Copy link
Contributor

I think I've run into another occurrence of this. Details in https://github.com/matrix-org/element-web-rageshakes/issues/22198.

@daniellekirkwood
Copy link
Contributor

I think we've had several more reports of this (i've had a few people in my DMs this past week)

@Johennes does that mean there's a new, different issue, or this one is getting worse (and needs to be changed from occasional)?

@Johennes
Copy link
Contributor

It's hard to say since we don't know the actual root cause. It might as well be something new or just the old issue increased in frequency by a tangential change.

@Johennes
Copy link
Contributor

We chatted this through in the WAT weekly today and we'll not escalate fixing this. We have other urgent work that is aligned with company-wide strategy this week and are generally not confident in making changes around the threads model without the refactoring and test suite mentioned in #24392. So this is, sadly, gated behind that work unless somebody would like to step in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Threads A-Timeline O-Occasional Affects or can be seen by some users regularly or most users rarely S-Critical Prevents work, causes data loss and/or has no workaround T-Defect X-Release-Blocker
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants