matrix-org / sygnal

Sygnal: reference Push Gateway for Matrix
Apache License 2.0
155 stars 140 forks source link

iOS VOIP sometimes notifications do not work if app is not open/in background #322

Closed Av164 closed 1 year ago

Av164 commented 1 year ago

iOS VOIP sometimes notifications do not work if app is not open/in background

Steps to reproduce the behavior:

  1. Make sure sygnal is set up properly for VOIP
  2. Make a VOIP call to your iOS device

Expected behavior VOIP notifications should get triggered and the phone should ring.

Smartphone (please complete the following information):

Additional context The app I am using is built on top of Element. I can see the .voip. app ID being called in the logs whenever I call someone and I have set push type for the .voip. app IDs to 'voip'.

reivilibre commented 1 year ago

I can see the .voip. app ID being called in the logs whenever I call someone and I have set push type for the .voip. app IDs to 'voip'.

Is this the Sygnal log? If so, it sounds like it may not be Sygnal's bug here?

It will likely need some research to find out which piece of software has the bug here; is it a Sygnal issue or an Element iOS issue (or possibly an issue with your custom app)?

I'm not an iOS developer and I'm afraid I'm not familiar with the restrictions. Though I believe for some aspects of notification handling that you need special consents from Apple; I don't know if this is one of those cases.

As a point of comparison, would you be able to check whether plain Element iOS from the Apple store has this problem?

DMRobertson commented 1 year ago

We're not easily able to investigate this. I put some links to relevant docs https://github.com/matrix-org/sygnal/issues/323#issuecomment-1259438550 which might help.

As a point of comparison, would you be able to check whether plain Element iOS from the Apple store has this problem?

This would be very helpful to know, but otherwise I suggest you talk to the Element iOS team for advice.

H-Shay commented 1 year ago

Closing in favor of #323