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

Errors in console likely linked to stuck unreads #25642

Closed
Tracked by #24392
andybalaam opened this issue Jun 21, 2023 · 8 comments
Closed
Tracked by #24392

Errors in console likely linked to stuck unreads #25642

andybalaam opened this issue Jun 21, 2023 · 8 comments
Assignees
Labels
A-Notifications O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@andybalaam
Copy link
Contributor

Steps to reproduce

  1. In a room, just browsing around
  2. Look in the console

Outcome

What did you expect?

Receipts to be sent without errors

What happened instead?

This error when some receipts are sent:

XHRPOST
https://matrix-client.matrix.org/_matrix/client/r0/rooms/!jSkNtsjcsXfCRJwGpp:matrix.org/receipt/m.read/$jdyMMc9_UwOUF1BSRcGekTmiGQvl5kIswe-juMwiRvw
[HTTP/2 400 Bad Request 29ms]
Error sending receipt 
Object { room: "!jSkNtsjcsXfCRJwGpp:matrix.org", error: M_INVALID_PARAM }
rageshake.ts:74:27
    fnName rageshake.ts:74
    instrumentConsole instrument.js:113
    methodFactory logger.ts:48
    sendReadReceipt TimelinePanel.tsx:1193
    TimelinePanel_TimelinePanel TimelinePanel.tsx:1163
    updateReadReceiptOnUserActivity TimelinePanel.tsx:1018
    componentDidMount TimelinePanel.tsx:325
    React 2
    unstable_runWithPriority scheduler.production.min.js:18
    React 4
    unstable_runWithPriority scheduler.production.min.js:18
    React 6
    setState MatrixChat.tsx:415
    viewRoom MatrixChat.tsx:978
    MatrixChat MatrixChat.tsx:695
    invokeCallback dispatcher.ts:120
    MatrixDispatcher dispatcher.ts:97
    sentryWrapped helpers.js:92
    (Async: setTimeout handler)
    _wrapTimeFunction trycatch.js:107
    dispatch dispatcher.ts:170
    viewRoom RoomViewStore.tsx:507
    onDispatch RoomViewStore.tsx:263
    invokeCallback dispatcher.ts:120

I've lost the response error message, but it was a 400 that said that the message was not in that thread.

I also see some messages like this in some rooms:

EventTimelineSet.addEventToTimeline: Ignoring event=$FIqf0uHppifN7Zgp9XWcrOEiXGhnKi58r1YK9F73Or0 that does not belong in timeline=!NasysSDfxKxZBzJJoE:matrix.org:2023-06-21T11:06:17.713Z timelineSet(threadId=undefined)

Rageshake (reported for a different issue): https://github.com/matrix-org/element-web-rageshakes/issues/21938

Operating system

No response

Browser information

No response

URL for webapp

No response

Application version

No response

Homeserver

No response

Will you send logs?

Yes

@weeman1337 weeman1337 added S-Minor Impairs non-critical functionality or suitable workarounds exist O-Occasional Affects or can be seen by some users regularly or most users rarely A-Notifications labels Jun 21, 2023
@t3chguy
Copy link
Member

t3chguy commented Jul 25, 2023

I believe this has been fixed

@t3chguy t3chguy closed this as completed Jul 25, 2023
@t3chguy t3chguy self-assigned this Jul 25, 2023
@andybalaam
Copy link
Contributor Author

I do still get this message in my console, lots of times. Not sure whether that is expected?

@t3chguy
Copy link
Member

t3chguy commented Aug 2, 2023

@andybalaam definitely not expected. Please send logs as well as corresponding context from devtools > Notifications debug

@t3chguy t3chguy reopened this Aug 2, 2023
@andybalaam
Copy link
Contributor Author

andybalaam commented Aug 3, 2023

Logs in https://github.com/matrix-org/element-web-rageshakes/issues/22359 show lots of "Ignoring event" messages. This was from a restart of the app. I don't see any stuck unreads as far as I know.

@andybalaam
Copy link
Contributor Author

Here is the notifs debug from a room that is mentioned in the error message:

image

@andybalaam
Copy link
Contributor Author

^ That room contains no threads. It's a DM.

@andybalaam
Copy link
Contributor Author

Here is an anonymised timeline from that room:
timeline.txt

@t3chguy
Copy link
Member

t3chguy commented Aug 4, 2023

I do still get this message in my console, lots of times. Not sure whether that is expected?

I thought you meant errors like Object { room: "!jSkNtsjcsXfCRJwGpp:matrix.org", error: M_INVALID_PARAM } - these are the ones which are signs of stuck notifications

The other ones are fine, they just mean we're not adding a reaction/relation to the timeline but still applying it in aggregate.

@t3chguy t3chguy closed this as completed Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Notifications O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

3 participants