Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Check subscription verification immediately after getting a new push token #7237

Closed
jonalmeida opened this issue Jun 4, 2020 · 1 comment
Labels
🌟 feature New functionality and improvements <push> Components: concept-push, lib-push-adm, lib-push-firebase, feature-push, feature-sendtab 🔬 Research Problems without a solution that need research.

Comments

@jonalmeida
Copy link
Contributor

jonalmeida commented Jun 4, 2020

When we get a push token, we wait to check our subscriptions when we next re-open the app (with initialize) and we haven't done a check recently (~24 hours). We could consider doing this check immediately after updating the token in the native layer in onNewToken.

I'm a little unsure if we should do this now. If we can verify the system works as expected after #7143 is fixed, we can consider doing this after the Fenix release.

We do however have our VerificationDelegate to avoid us DDOS-ing our push servers if something goes wrong with our FxA servers, so there is safety there.

┆Issue is synchronized with this Jira Task

@jonalmeida jonalmeida added 🌟 feature New functionality and improvements 🔬 Research Problems without a solution that need research. <push> Components: concept-push, lib-push-adm, lib-push-firebase, feature-push, feature-sendtab labels Jun 4, 2020
@data-sync-user data-sync-user changed the title Check subscription verification immediately after getting a new push token FNX3-22749 ⁃ Check subscription verification immediately after getting a new push token Aug 5, 2020
@data-sync-user data-sync-user changed the title FNX3-22749 ⁃ Check subscription verification immediately after getting a new push token FNX2-18263 ⁃ Check subscription verification immediately after getting a new push token Aug 5, 2020
@st3fan st3fan changed the title FNX2-18263 ⁃ Check subscription verification immediately after getting a new push token Check subscription verification immediately after getting a new push token Aug 5, 2020
@cpeterso
Copy link
Contributor

Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1796427

Change performed by the Move to Bugzilla add-on.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🌟 feature New functionality and improvements <push> Components: concept-push, lib-push-adm, lib-push-firebase, feature-push, feature-sendtab 🔬 Research Problems without a solution that need research.
Projects
None yet
Development

No branches or pull requests

2 participants