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

Notification 'red' badge count is flawed in E2EE rooms #17223

Open
Tracked by #3363
ara4n opened this issue May 10, 2021 · 6 comments
Open
Tracked by #3363

Notification 'red' badge count is flawed in E2EE rooms #17223

ara4n opened this issue May 10, 2021 · 6 comments
Labels
A-E2EE A-Notifications O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Z-GetYourUpdates

Comments

@ara4n
Copy link
Member

ara4n commented May 10, 2021

You can only correctly calculate badge counts for mentions & keyword push rules if you decrypt every message in an E2EE room on the client. Currently if there is a gappy sync (due to the app being offline overnight, or the client failing to keep up with the server) then messages deliberately get dropped. Therefore the 'red' badge count for missed notifications is wrong (and the NotifPanel is wrong) whenever the app awakes in the morning - which is when you want the badge counts to be the most correct.

Instead we should spider the E2EE rooms in the background in order to calculate the right notification count. Luckily we have the code to do this already for Seshat - but we need to actually hook it up on both Web & Desktop so that it triggers badge count calculations as it spiders, so we at last get the correct badge counts.

cc @manuroe and @bmarty for iOS & Android, as the same problem exists there (but is less pronounced given gappy syncs are somewhat less common given people tend not to turn off or disconnect their phones as often as they do their laptops).

cc @gsouquet as this overlaps somewhat with his decrypt-on-demand work in matrix-org/matrix-js-sdk#1684

@ara4n ara4n added the T-Defect label May 10, 2021
@jryans jryans added A-Notifications P2 S-Minor Impairs non-critical functionality or suitable workarounds exist labels May 10, 2021
@ara4n
Copy link
Member Author

ara4n commented May 10, 2021

(is it really minor that your notifications plain lie about whether you've been notified or not?)

@babolivier
Copy link
Contributor

babolivier commented May 26, 2021

I also disagree with this bug being categorised as minor. Since I close my work Element session when I'm not working (which I assume a fair amount of people using Element for work would do), this means that I'll need to go through each of my encrypted rooms manually to make sure I haven't missed a notification at least once a week (after the weekend - I got hit by it on Monday) and each time I come back after some time off. IMHO this doesn't paint a great picture of Element in terms of reliability if using it for work means it's likely to drop notifications on the floor at least once a week.

@MadLittleMods MadLittleMods added A-E2EE P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround and removed P2 S-Minor Impairs non-critical functionality or suitable workarounds exist labels Sep 24, 2021
@MadLittleMods
Copy link
Contributor

(from #9069) Also related to matrix-org/matrix-spec-proposals#1796

@andybalaam andybalaam added O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience and removed P1 labels Apr 27, 2022
@germain-gg germain-gg added the Z-AccountResurrection Delight - cycle label Aug 23, 2022
@germain-gg germain-gg removed the Z-AccountResurrection Delight - cycle label Nov 17, 2022
@richvdh
Copy link
Member

richvdh commented Mar 16, 2023

Instead we should spider the E2EE rooms in the background in order to calculate the right notification count.

For links: this also came up in the context of discussions around MSC3952 (matrix-org/matrix-spec-proposals#3952 (comment)).

@richvdh
Copy link
Member

richvdh commented Mar 16, 2023

Closely related: #6874.

@MadLittleMods
Copy link
Contributor

MadLittleMods commented Apr 17, 2023

👍 Very annoying to have my Element Web missing notifications which I know they were there from over the weekend. Subtle/insidious way to miss notifications.

Element Web that was running the whole time but was refreshed and is now missing red badge for the Moderation Internal room Element desktop that ran the whole time and shows mentions for Moderation Internal
element-room-list-missing-notifications-for-moderation-internal element-room-list-with-notifications-for-moderation-internal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE A-Notifications O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Z-GetYourUpdates
Projects
None yet
Development

No branches or pull requests

9 participants