-
Notifications
You must be signed in to change notification settings - Fork 752
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
Push Notifications not working or Inadequate Documentation on setting up FCM GCM push with own Sygnal server #6115
Comments
Should note that the tools/test_push.sh did seem to work and gave: |
I guess NOTIFICATIONS simply do not work at all on Android/mobile? |
without knowing your entire setup it's hard to diagnose but I can confirm push notifications are working in Element Android (and in my personal project which hosts its own sygnal server) To double check
|
@ouchadam I did a lot more testing and decided the issue is likely with dendrite<-->element-android not getting "PUSH RULES". This is because I noticed in Advanced Settings-Notification Targets that pressing Push Rules showed a BLANK page. Where is the code to negotiate population of those PUSH RULES into the element-android app?
So the app itself seems can receive and I been guessing PUSH RULES aren't propagated to Client from Dendrite server. (PLEASE NOTE that REGISTRATION on my dendrite homeserver is disabled, so I MANUALLY CREATE users from command-line and so I do NOT do the "registration" process through the CLIENT/element-android app with dendrite, I just use "sign-in".) Not sure if this would be a bug with Client/Element or with Dendrite. OH and ALSO, to answer your questions:
|
@ouchadam Can you confirm in your personal project where notifications work, are you using DENDRITE with sygnal or are you using Synapse instead? Also if indeed you are using DENDRITE, did you create the user you're logged into element android, did you create it from command-line on Dendrite server, or did you actually do the "Registration" process in element app to register user into your dendrite? thanks! |
@ouchadam This has basically been reconfirmed by me and I think now by others as well. In fact notifications can be made to work by going to the settings of a room or direct message room and changing notifications to "none" and then back to "all". After that you will see the Push Rules will not be blank anymore but populated in Advanced Settings>Notifications>Push Rules After that NOTIFICATIONS started working, before the push rules populated, they did not work. |
thanks for the information 👍
unfortunately I'm running a synapse homeserver, it's worth noting that sygnal itself is not directly tied to a homeserver, it's a simple http proxy that will forward to the various supported push mechanisms
this isn't the case for debug builds pointing to matrix.org, the package id and signing key are used in conjunction to prove debug authenticity
As part of the sync handlers UserAccountDataSyncHandler New events from the sync are actioned by the ProcessEventForPushTask (FCM push notifications trigger a sync) This issue sounds like two possible scenarios
|
@ouchadam Actually I can try do double check, because all of my builds are debug builds so far, and I think some of them I had left with matrix.org pushgateway and with matrix.org's github included google json file and I think it works. I also built with my own pushgatway using same debug key, i don't know if the debug key is some generic one or one included in github repo, but notifications are working with my sygnal push-gateway via my custom build using same debug key. So I'm not sure if FCM cares to match the apk signature. (Just think that you may want to deploy same app with different signatures, to different app stores etc, and if FCM was happy enough with apk containing proper googel-json FCM token and senderid etc, then you would not have to create a separate FCM entry for each different distribution channel and apk signatures. Anyway maybe I can retry sometime to really confirm, but busy with a lot of other things so just can't now.) I think someone did a recent PULL-Request to Dendrite to fix the Push Rules issue, see: |
The issue on Dendrite and using |
@S7evinK Yea without the matrix-org/dendrite#2484 patch, I was able to populate Push Rules by the mentioned workaround of going to any room and settings on that room and in Notifications changing Notify me for to "none" and then back again to "all messages", and after this the Push Rules will appear in the client finally, instead of all blank. The above is less hassle than signing out and back in again and having to re-import encryption keys etc etc. |
I am wonderring whether this issue perfectly fixed or not? |
Steps to reproduce
But I can NOT get any notifications on android phone.
Outcome
What did you expect?
I expect to get some notification on android phone when app gets a new message.
What happened instead?
Nothing.
I don't see any error messages in sygnal.log or in dendrite log when I'm sending the message and no notification comes up.
Your phone model
Samsung Note9
Operating system version
Android 8.1
Application version and app store
Element 1.4.16-dev olm 3.2.10 matrix sdk 1.4.16
Homeserver
Dendrite github
Will you send logs?
Yes
The text was updated successfully, but these errors were encountered: