mozilla-mobile / firefox-ios

Firefox for iOS
Mozilla Public License 2.0
12.15k stars 2.91k forks source link

Refactor Notification Service extention to not use the tab queue #14304

Closed tarikeshaq closed 1 year ago

tarikeshaq commented 1 year ago

The way the notification service currently works is the following:

The above is mostly okay - however, we have a mechanism to retrieve missed send tabs by polling (in case the push notification never arrived) by passing the tabs in the queue and polling, we have two ways to get send tabs and it's possible for them to race - see https://github.com/mozilla-mobile/firefox-ios/issues/13136

Opening this ticket for myself to change the flow so that the Notification Service does not need the tab queue, and instead sets the polling pref to indicate that the app should hit the FxA server when it wakes up to get the tab.

┆Issue is synchronized with this Jira Task

tarikeshaq commented 1 year ago

Closed in https://github.com/mozilla-mobile/firefox-ios/pull/14305