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

double notifications #213

Open
nanopone opened this issue Apr 13, 2023 · 5 comments
Open

double notifications #213

nanopone opened this issue Apr 13, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@nanopone
Copy link

Describe the bug
when receiving a message, i get the notification for it, but also a generic "New message!" notification alongside it

To Reproduce
Steps to reproduce the behavior:

  1. have notifications turned on
  2. receive a message

Expected behavior
i only expect to receive one notification for each message

Screenshots
B29C3011-EDF2-4D69-A367-F9A70949920E

Details (please complete the following information):

  • Siskin Version: 7.3.3
  • iOS version: 16.3.1
  • iPhone model: iphone 13 mini
@nanopone nanopone added the bug Something isn't working label Apr 13, 2023
@hantu85
Copy link
Contributor

hantu85 commented Apr 13, 2023

Siskin does not generate push notifications for incoming messages when the client is in the background (disconnected). Those notifications are generated by your XMPP server and Siskin just displays them. Notifications that are generic "New message!" do not contain any message but are sent by your XMPP server. If your server sends notifications with content/message, then the proper message is displayed.

We are aware that some XMPP servers send notifications for any incoming XMPP stanza or message (even if there is no body in the message), but that is in our opinion violation of the XEP-0357: Push Notifications and should be solved on the server side.

In future versions we may decide to filter notifications on the client side, however, I do not think this is how it should be done.

@t0maz
Copy link

t0maz commented Aug 24, 2024

The issue is still there. No such problem with Monal client. The problem is even more important as after opening Siskin and reading the new message in the chat the „New message!” notification do not disappears like the one with message content. This cause that Siskin app keep red counter with number of unreaded chats/messages while there is no more unreaded ones in the app. This confuse users very much. To get rid this bogus counter of unreaded chats the system notification have to be cleared manually in the iOS notifications.
I do really hope it can be fixed in future release as it’s hard to teach / explain to non techy users how to workaround it.
Thanks!

@hantu85
Copy link
Contributor

hantu85 commented Aug 27, 2024

Some XMPP servers do not send message body nor any other identifiers allowing us to match received notification to the message received later on when app reconnects to the XMPP server. Due to the approach chosen by us, to not reconnect to the server to fetch all remaining messages on each received push notification, it is impossible to fix this issue.

The solution is to use a different XMPP server that supports sending additional details (ie. in encrypted form). According to my knowledge it is possible with Tigase XMPP Server and with Prosody.

@t0maz
Copy link

t0maz commented Aug 27, 2024

Many thanks for clarification.
That’s great news!
We do use latest release of Prosody server 0.12.4.
Can you please tell what needs to be changed in server configuration to avoid bogus/empty push notifications with Siskin client or where such information can be found?
Thank you

@hantu85
Copy link
Contributor

hantu85 commented Aug 27, 2024

If I recall correctly, https://modules.prosody.im/mod_cloud_notify_encrypted needs to be configured and enabled.

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

3 participants