Push notifications are not delivered to IOS app

pavloburykh commented 3 months ago

Reproducing on IOS 17.5.1, iPhone 15

Preconditions: User A - message sender (any device), User B - PN receiver, IOS device

Steps:

User B enables PN notifications
User B backgrounds app
User A sends messages to any common chat with User B (1-1, group, community channel)
See if User B received PN about new message

Actual result: PN is not received by User B

Andr_PN_sender.zip IOS_PN_receiver.zip

https://github.com/user-attachments/assets/b004c7a0-fc8e-4615-97ae-24499ea8eb94

Additional Information

Status version: release 2.30
Operating System: iOS

pavloburykh commented 3 months ago

UPDATE: sometimes PNs go through. @mariia-skrypnyk and @churik were able to receive some PNs.

cammellos commented 3 months ago

I have investigated the issue:

What looks like, in at least one instance:

1) The device was correctly registered with the push notification server 2) The sender, queried the push notification server for informations, via lightpush. The message made it to the server 3) The server replied to the message 4) The reply was never received by the sender, therefore the push notification wasn't sent

Looking at kibana's logs, I can see that no filter node has pushed that message to the right client:

https://kibana.infra.status.im/app/discover#/?_g=(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-15h,to:now))&_a=(columns:!(),filters:!(),index:ffcc22b0-0116-11ed-9719-cdd3b483481c,interval:auto,query:(language:kuery,query:'%220x9f5d30ee6c008d365e20014c749bf3194d960467edaa71f89880721abf635180%22%20AND%20kv_raw:filter'),sort:!(!('@timestamp',desc)))

This is the missing message. The peer id of the sender is "16U*hhQfss"

You can see a couple of minutes later, a successful push:

https://kibana.infra.status.im/app/discover#/?_g=(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-15h,to:now))&_a=(columns:!(),filters:!(),index:ffcc22b0-0116-11ed-9719-cdd3b483481c,interval:auto,query:(language:kuery,query:'%220x34304e1a4764bb92c5ccfec8e59a56e85b99651fb70d88c956799d2a5103a139%22%20and%20kv_raw:filter'),sort:!(!('@timestamp',desc)))

But now looking at the client log, it looks like they never registered for that particular topic, so i'll start from scratch investigating.

cammellos commented 3 months ago

DEBUG[08-23|10:46:10.968|github.com/status-im/status-go/protocol/pushnotificationclient/client.go:810]                                       checking if querying necessary 
DEBUG[08-23|10:46:10.968|github.com/status-im/status-go/protocol/pushnotificationclient/client.go:815]                                       querying info 
DEBUG[08-23|10:46:10.989|github.com/status-im/status-go/wakuv2/common/filter.go:108]                                                         filters install                          id=3867b9e1d9cd67826de7426adbafac0aa6d9ec80374057c478ee92f35971477f
DEBUG[08-23|10:46:10.989|github.com/status-im/status-go/vendor/github.com/waku-org/go-waku/waku/v2/api/filter/filter_manager.go:111]         adding filter                            filter-id=3867b9e1d9cd67826de7426adbafac0aa6d9ec80374057c478ee92f35971477f
DEBUG[08-23|10:46:10.989|github.com/status-im/status-go/vendor/github.com/waku-org/go-waku/waku/v2/api/filter/filter_manager.go:124]         existing pubsubTopic batch               agg-filter-id=dcd3c805-b342-46be-a406-ebc445802ae1 topic=/waku/2/rs/16/32
DEBUG[08-23|10:46:10.989|github.com/status-im/status-go/vendor/github.com/waku-org/go-waku/waku/v2/api/filter/filter_manager.go:145]         adding to existing pubsubTopic batch     agg-filter-id=dcd3c805-b342-46be-a406-ebc445802ae1 content-filter="{ pubsubTopic: /waku/2/rs/16/32, contentTopics: [ /waku/1/0x25a24866/rfc26 ] }" batch-size=3
DEBUG[08-23|10:46:10.989|github.com/status-im/status-go/protocol/transport/filters_manager.go:441]                                           registering filter for                   namespace=filtersManager chatID=contact-discovery-950 type=partitioned topic=0x25a24866
DEBUG[08-23|10:46:10.991|github.com/status-im/status-go/protocol/common/message_segmentation.go:47]                                          message segmented                        segments=1
DEBUG[08-23|10:46:11.099|github.com/status-im/status-go/wakuv2/common/filter.go:108]                                                         filters install                          id=8a08fc1993745515933e0ec0044e4ac3f8a08933e0da387c3bf5ae82c7d7ead4
DEBUG[08-23|10:46:11.099|github.com/status-im/status-go/vendor/github.com/waku-org/go-waku/waku/v2/api/filter/filter_manager.go:111]         adding filter                            filter-id=8a08fc1993745515933e0ec0044e4ac3f8a08933e0da387c3bf5ae82c7d7ead4
DEBUG[08-23|10:46:11.099|github.com/status-im/status-go/vendor/github.com/waku-org/go-waku/waku/v2/api/filter/filter_manager.go:124]         existing pubsubTopic batch               agg-filter-id=dcd3c805-b342-46be-a406-ebc445802ae1 topic=/waku/2/rs/16/32
DEBUG[08-23|10:46:11.099|github.com/status-im/status-go/vendor/github.com/waku-org/go-waku/waku/v2/api/filter/filter_manager.go:145]         adding to existing pubsubTopic batch     agg-filter-id=dcd3c805-b342-46be-a406-ebc445802ae1 content-filter="{ pubsubTopic: /waku/2/rs/16/32, contentTopics: [ /waku/1/0x815b41c6/rfc26 ] }" batch-size=4
DEBUG[08-23|10:46:11.099|github.com/status-im/status-go/protocol/transport/filters_manager.go:574]                                           registering filter for                   namespace=filtersManager chatID=95d7f2cdb31b8b1abbdd528aaf97d3a9c7a5285b1a184e6ff5d1a52a7b6e7b9889b38d120795e068ead5f5c27389f9696ec09ce06f1765bafd211e065ca37ce4 type=public ContentTopic=0x815b41c6 PubsubTopic=
DEBUG[08-23|10:46:11.100|github.com/status-im/status-go/protocol/common/message_sender.go:805]                                               sent-message: public message             recipient=[] messageID=0x17b3a4d64ebe9f38f5eb7fe8272feb1018d8c3ba5f63089f88b8974fe02c021b contentType=PUSH_NOTIFICATION_QUERY messageType=public hashes=[0xf7bbddd9f231f2601012b95ccc939b7dae76f5c2a5747da50c51f006588e7ca0]
DEBUG[08-23|10:46:11.100|github.com/status-im/status-go/wakuv2/common/filter.go:249]                                                         processing message: decrypted            envelopeHash=0xf7bbddd9f231f2601012b95ccc939b7dae76f5c2a5747da50c51f006588e7ca0
INFO [08-23|10:46:11.100|github.com/status-im/status-go/vendor/github.com/waku-org/go-waku/waku/v2/api/publish/message_sender.go:115]        publishing message via lightpush         envelopeHash=0xf7bbddd9f231f2601012b95ccc939b7dae76f5c2a5747da50c51f006588e7ca0 pubsubTopic=/waku/2/rs/16/32 contentTopic=/waku/1/0x815b41c6/rfc26 timestamp=1,724,409,971,099,899,579

Looking at this message now:

HASH=0xf7bbddd9f231f2601012b95ccc939b7dae76f5c2a5747da50c51f006588e7ca0
CONTENT_TOPIC=/waku/1/0x815b41c6/rfc26

The message was received by a push notification server:

https://kibana.infra.status.im/app/discover#/context/ffcc22b0-0116-11ed-9719-cdd3b483481c/C9HXfpEBaE8Vq-sg_ypD?_g=(filters:!())&_a=(columns:!(),filters:!(('$state':(store:appState),meta:(alias:!n,disabled:!f,index:ffcc22b0-0116-11ed-9719-cdd3b483481c,key:logsource,negate:!f,params:(query:node-01.do-ams3.notify.prod),type:phrase),query:(match_phrase:(logsource:node-01.do-ams3.notify.prod)))),predecessorCount:10,successorCount:5)

cammellos commented 3 months ago

The response, was sent:

 INFO [08-23|10:46:10.404|github.com/status-im/status-go/wakuv2/waku.go:1016] publishing message via relay envelopeHash=0xbbd9ab3762e6cee821a5dddbf240018e43d96c4ed8786c44b519bcc9ac19b408 pubsubTopic=/waku/2/rs/16/32 contentTopic=/waku/1/0x25a24866/rfc26 timestamp=1,724,409,970,404,010,117 peerCnt=15

Notice the time:

10:46:10.404

And the content topic:

/0x25a24866

If we go back to the client:

10:46:10.989  adding to existing pubsubTopic batch     agg-filter-id=dcd3c805-b342-46be-a406-ebc445802ae1 content-filter="{ pubsubTopic: /waku/2/rs/16/32, contentTopics: [ /waku/1/0x25a24866/rfc26 ] }" batch-size=3
10:46:10.989 registering filter for                   namespace=filtersManager chatID=contact-discovery-950 type=partitioned topic=0x25a24866

That's when we register the filter.

10:46:12.569      ticker hit, hence subscribing            agg-filter-id=dcd3c805-b342-46be-a406-ebc445802ae1 batch-size=4 agg-content-filter="{ pubsubTopic: /waku/2/rs/16/32, contentTopics: [ /waku/1/0x7bcb873b/rfc26/waku/1/0xbd4cfa16/rfc26/waku/1/0x25a24866/rfc26/waku/1/0x815b41c6/rfc26 ] }"

But then 2 seconds later it's actually tried to register

10:46:12.889subscription successful                  pubSubTopic=/waku/2/rs/16/32 contentTopics="[/waku/1/0x7bcb873b/rfc26 /waku/1/0xbd4cfa16/rfc26 /waku/1/0x25a24866/rfc26 /waku/1/0x815b41c6/rfc26]" peer=16Uiu2HAmMhsfGvYqkAryRe8WZnKHJUhKRbni7hseGpRy174hRaZv

So the timeline is:

1) Client schedule registering filter on content topic A 2) Client pubilshes message 3) Server receives message and replies on content topic A 4) Client actually registers the filter on content topic A

Message would be received between 3 & 4, but filter is not pushed as the client hasn't yet register the filter.

You can see similar examples from the logs, where we query store nodes just after scheduling register a filter, but we receive messages too early.

https://github.com/status-im/status-go/pull/5440 introduced the ticker.

Before the issue was probably already present (registering a filter was probably asynchronous, although I think on v1 it was sync), but most likely was never picked up as the window was smaller.

Potential solutions:

we mitigate but reducing the timing, but the window would still be present
we delay the sending until the filter is registered, that would add a concept of dependency to sending a message (i.e sending message X depends on having registered message Y)
we refetch from store node once a new filter is subscribed, for the period between scheduling the filter and once the filter has been actually registered, with some padding of course to accomoddate the timing. (there's still a window where the message would not be on the store node, nor the message has been pushed by the filter though).

Either ways, this looks like a fairly critical message reliability issue, lowering the batch ticker (it's currently I believe 5 seconds, which is quite high) is probably the quickest to lower the possibility of that happening.

cc @churik @fryorcraken @chaitanyaprem @richard-ramos @ilmotta and thank you @pavloburykh for all the logs

richard-ramos commented 3 months ago

As a way to not block release, I'd say that we can reduce the ticker length to a length which fast enough to make this issue potentially not happen (maybe something like 500ms or less?).

Then we can try work out a proper solution. @chaitanyaprem and I had talked before about this possible window on which messages would not be received due to the ticker and doing the storenode request could be a good solution indeed, taking into account that there might be a delay for the message to arrive to the storenode.

Nwaku storenodes will only accept messages with a timestamp +-20s its current time, so based on this, when we register a filter and the ticker is triggered, we asynchronously wait 20s, and then send a store request. What do you think, @cammellos and @chaitanyaprem?

churik commented 2 months ago

Improved in https://github.com/status-im/status-go/commit/4a18c85c3c1d58ea6c8493c46bfd2ed5772b1386 as a temp measure; the issue requires more investigation / actions, so postponed to 2.31

chaitanyaprem commented 2 months ago

Nwaku storenodes will only accept messages with a timestamp +-20s its current time, so based on this, when we register a filter and the ticker is triggered, we asynchronously wait 20s, and then send a store request. What do you think, @cammellos and @chaitanyaprem?

I am wondering if doing a store query after 20secs would help, because current missing messages periodic store query anyways happens every 20seconds. Which means anyways worst-case these messages would be fetched in next store query isn't it (unless i am missing something)?

pavloburykh commented 2 months ago

Issue is still valid for me. @status-im/mobile could you please also re-check if you are receiving push notifications on IOS?

VolodLytvynenko commented 2 months ago

still valid. No pushes on IOS.

Devices:

iPhone 11 Pro Max, IOS 17

mariia-skrypnyk commented 2 months ago

@pavloburykh checked on 19/09 nightly on iOS - no PN .

iPhone 13, iOS 17

churik commented 2 months ago

I still have notifications working on release 2.30 and in PRs (enabled from profile - and from onboarding)

IPhone 14 Pro, IOS 17.6.1

chaitanyaprem commented 2 months ago

referring issue being tracked for go-waku https://github.com/waku-org/go-waku/issues/1211 we thought after reducing the ticker the issue has not happened.

but looks like the fix has not been propagated correctly it into the develop branch which is why push notifications must be failing. let me create a PR with fix pushed to develop branch as well and we can verify the same.

This is why @churik is not facing any issues as she is testing on 2.30 whereas others might be testing on develop branch.

chaitanyaprem commented 2 months ago

pushed a fix in https://github.com/status-im/status-mobile/pull/21297

@pavloburykh @churik can you please get this revalidated and confirm if the issue still happens.

86doteth commented 1 month ago

i havent gotten any push notification in 2.30 release. tested it just now in dms nothing arrives. iphone xs @chaitanyaprem @churik

edit: its worse actually. after 5 hours decided to open the mobile app and then had to wait for about 3-4 minutes before any new messages appeared inside the app including the DM i sent from another account. unfortunately this is how its been for the past months: to just load any new messages i need to wait for multiple minutes upon opening the app every time. there doesnt seem to be any prioritization either i can wait just as long for any dm to appear as i can for any random channel in a community having unread messages. i hope there are issues addressing these loading times bc i just have status inu status and like 3 sparse dm chats

churik commented 1 month ago

@ilmotta I believe this is related to the changes in login process, but frankly I'm not sure how we should log and track such performance issues. If user has several communities / 1-1 chats / group chats that indeed might lead to some delay after login in starting fetching messages, and afaik we're aware of that. Should I log this separately?

ilmotta commented 1 month ago

I believe this is related to the https://github.com/status-im/status-mobile/pull/20729

Could be, but I don't see how at the moment. If we could reproduce this issue reliably it would be pretty easy to create a new build to login in the old way and check if it's related to the issue you mentioned. But basically we just moved one step from the messenger to happen after login, but the time to setup is 100% the same.

But I think what we did in https://github.com/status-im/status-mobile/pull/20173 was a necessary step. We can't mess up the login and onboarding step. Based on past investigation, a big slice of the cost to set-up filters during login is related to the cost of decrypting communities. Assuming many don't want to use communities and chats and might just want to use the wallet or something else, login should happen as fast as possible.

We have a few ways to improve the login:

Optimize the code setting up filters and processing community chats.
Improve UX to take in consideration that, realistically, after a user logs in we can't guarantee chats will be immediately available.

I remember that we still had room left to optimize the way we process community chats. This is the problematic function https://github.com/status-im/status-go/blob/77ef8f1fb7e6ee08b6f09e4e19cf7b4b5f062a16/protocol/messenger.go#L1769

to just load any new messages i need to wait for multiple minutes upon opening the app every time. there doesnt seem to be any prioritization either i can wait just as long for any dm to appear as i can for any random channel in a community having unread messages.

This is a terrible experience @86doteth...

Should I log this separately?

I would say so. The performance problem to initialize the messenger is well known to us, we are just now seeing that it can get worse quickly depending on the type/amount of data the user has.

churik commented 1 month ago

I'll try to investigate the timing and report it as a separate performance issue. Basically, the more communities you have, the later you'll start to receive the messages.

And yes, it was not a question of the necessity of changing the login; rather, this problematic function is now handled after login, which gives the impression of a significant delay (to my understanding)

Thank you for reply, @ilmotta

pavloburykh commented 1 month ago

@

I'll try to investigate the timing and report it as a separate performance issue. Basically, the more communities you have, the later you'll start to receive the messages.

And yes, it was not a question of the necessity of changing the login; rather, this problematic function is now handled after login, which gives the impression of a significant delay (to my understanding)

Thank you for reply, @ilmotta

@churik I have logged a separate issue here https://github.com/status-im/status-mobile/issues/21358 cc @ilmotta

churik commented 1 month ago

we're closing this one as the problem with performance moved here and so far we're receiving IOS push notifications in develop and in release. Will open a new one if it will reproduce again

86doteth commented 1 month ago

we're closing this one as the problem with performance moved here and so far we're receiving IOS push notifications in develop and in release. Will open a new one if it will reproduce again

with release i guess you mean 2.31? bc i havent gotten any push notifs on 2.30

churik commented 1 month ago

@86doteth I'm not sure if I asked you (can't see from the comments here), but there are some things to consider: 1) did you upgrade from 1.20.6? If yes, you need to re-enable Push notifications in your profile, it won't work without that (it is known) 2) please, check if status is allowed app in your Focus settings in the app.

Push notifications were working for me on release 2.30. Please let me know if the issue persist.

86doteth commented 1 month ago

@churik

i upgraded from the testflight to 2.30. before testflight i was on 1.20.6 and removed the old app before installing testflight. and the push notifications have been enabled inside status settings all along.
dont see app specific settings under Focus in ios settings but status does have all push notifications enabled in ios settings.

churik commented 1 month ago

@86doteth please, try to re-enable it in Profile, re-login and see if it helps. Thank you for your patience, and apologies for these workarounds.

86doteth commented 1 month ago

@churik i disabled push notifications in profile then re-enabled it. then force-closed the app. then re-opened the app, logged in, and then left the app. then sent a test message from another account to mobile and 4 mins later havent received any push notification yet (and havent opened the app since)

churik commented 1 month ago

reopened it with no-steps, as we're unable to reproduce it.

86doteth commented 1 month ago

@churik maybe more parts of my setup that are useful:

ive sent the test message from another account on desktop
my account is a member of status community+status inu
this account also has at least two other desktop instances on which i logged in in recent weeks (one today even but closed it before doing the test)

86doteth commented 1 month ago

another thing i noticed thats hopefully useful

every time i do the test, i.e. send a message from a desktop account to mobile account while keeping mobile app closed, the message status on desktop just stays on sent and only changes to delivered after step 3 of the following:

open the mobile app
wait for the chat/communities section to become unfrozen (at least 30-50 seconds)
see recent messages being loaded including the dm from desktop (can take minutes some times)

so idk if this maybe messes up the push notification logic from kicking in if message doesnt get delivered status somehow?

i made sure in advanced settings to have store confirmations turned on (same results when its turned off). further settings are both light client and peer confirmations turned on. also tried with vpn turned on and off but same results.

also i tried the same test but this time while keeping both desktop and mobile chat screens open the whole time. i waited for about 110 secs on the mobile side until the mobile app crashed (regular occurrence after a few mins) while on desktop side the message status stayed on sent. then reopened the mobile app waited another 50 seconds and then the messages showed up while simultaneously on desktop the status changed to delivered.

one other thing ive observed is that messages sent from mobile never get a delivered status but only show sent even though their messages arrive fine on desktop

86doteth commented 3 weeks ago

well the one thing i havent tried before was removing and reinstalling the app and doing that somehow did the trick bc i now receive push notifications for dms. only odd thing about my setup i can think of is that i had the testflight app installed before 2.30 (and removed status v1 before installing testflight) and then iirc simply upgraded to 2.30 when it came bc i still had all message history from july onwards. maybe the testflight app still had some leftover config that messed things up in 2.30?

i couldnt verify if mentions work. 2.30 fresh install seems unable to fetch any communities, whether in discover section or ones sent in dm. besides its an account imported by sync so expected the communities i already joined to be visible but it says i havent joined any on the mobile install. so there were no communities in which i could try mentioning the account. if i tried anyway then nothing happens on mobile side

EDIT: i managed to finally fetch some communities on the mobile account and verified that direct mentions work. just not the ‘everyone’ mention when used from desktop, but i guess that one isnt implemented yet on mobile because typing @+everyone on mobile doesnt do anything

86doteth commented 3 weeks ago

btw why dont status push notifications on ios ever make any sound? its been the same on v1 as well. always checked my ios notifications settings and sound is always on. would really like sound to work so i dont have to watch my phone screen all the time when conversating

86doteth commented 3 weeks ago

aight ive made two issues for the problems in my last two comments:

communities not fetching on first login (after install): https://github.com/status-im/status-mobile/issues/21525
push notifications making no sound on ios: https://github.com/status-im/status-mobile/issues/21536

yall can close this issue now if you want as far as im concerned 👍

ilmotta commented 3 weeks ago

@86doteth We momentarily deprioritized push notifications (PNs) from 2.30 until 2.32 (to be released by the end of the year if all goes well). We are carefully listening for feedback in this area and we intend to attack some pesky PN problems for release 2.33.

Especially for a messaging app, there must exist an excellent PN solution and we want to do that while catering for privacy and battery life. At the moment, this feature is unstable (has been for a long long time) and we can't make guarantees about its quality and reliability.

communities not fetching on first login (after install)

Thanks for the detailed issue you created, that's quite helpful. The Discover Communities feature requires some improvements for sure and we are considering that in our priorities.

yall can close this issue now if you want as far as im concerned 👍

We will leave most PN issues open to help us define scope and priorities for release 2.33+ 👍🏼

Thank you!

cc @sunleos

86doteth commented 3 weeks ago

thanks for the informative replies @ilmotta

Especially for a messaging app, there must exist an excellent PN solution and we want to do that while catering for privacy and battery life.

agreed. would be cool if yall could collab with desktop team and make it possible to optionally have your own desktop instance serve as your personal push notification server. the server code is already in status-go so would be (in my perhaps too optimistic view) just a ui-addition in desktop and a new setting on mobile. aside from the huge privacy boost i think it also allows an ios user to subscribe to as many kind of push notifications as they want such as say all new messages in any community or channel and not just mentions and 1-1 chats like now

more on running your own push server, i.e. anonymous mode: https://github.com/status-im/specs/blob/master/docs/raw/push-notification-server.md?ref=our.status.im#anonymous-mode-of-operations

the server code already in status-go: https://github.com/status-im/status-go/blob/e53a5c968e2df809651f5894a2833c2e0a54b45a/protocol/pushnotificationserver/server.go

but maybe its harder than it looks, just thinking out loud here ;)

ilmotta commented 3 weeks ago

thanks for the informative replies @ilmotta

Especially for a messaging app, there must exist an excellent PN solution and we want to do that while catering for privacy and battery life.

agreed. would be cool if yall could collab with desktop team and make it possible to optionally have your own desktop instance serve as your personal push notification server. the server code is already in status-go so would be (in my perhaps too optimistic view) just a ui-addition in desktop and a new setting on mobile. aside from the huge privacy boost i think it also allows an ios user to subscribe to as many kind of push notifications as they want such as say all new messages in any community or channel and not just mentions and 1-1 chats like now

more on running your own push server, i.e. anonymous mode: https://github.com/status-im/specs/blob/master/docs/raw/push-notification-server.md?ref=our.status.im#anonymous-mode-of-operations

the server code already in status-go: https://github.com/status-im/status-go/blob/e53a5c968e2df809651f5894a2833c2e0a54b45a/protocol/pushnotificationserver/server.go

but maybe its harder than it looks, just thinking out loud here ;)

That is a pretty interesting idea @86doteth +1 Thanks a bunch

status-im / status-mobile

Push notifications are not delivered to IOS app #21120

Additional Information

Devices: