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

Stream group isActive so Notifications can be unsubscribed #1785

Open
nplasterer opened this issue Mar 21, 2025 · 0 comments
Open

Stream group isActive so Notifications can be unsubscribed #1785

nplasterer opened this issue Mar 21, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@nplasterer
Copy link
Contributor

nplasterer commented Mar 21, 2025

  • an app subscribes to a list of topics
  • push notification server listens to those topics for any new message
  • you get removed from that group next sync you process the commit and it sets the group to inactive but the only way you know the group is inactive is because is by going to the group and calling group.isActive() In a sync all the commit would get processed behind the scenes and no unsubscribe would be triggered?
  • The push notification server is still listening on this topic and sending push notifications for messages in that group that you can no longer decrypt. After receiving one push you can see the decryption error catch it and then unsubscribe but the user would still get one more push?

I see what you're saying. I guess we could add some sort of callback or hook as part of sync to notify the caller that they need to update their subscriptions. Then they could be unsubscribed before the next message comes in (edited)

@nplasterer nplasterer added the bug Something isn't working label Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant