Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Ensure that update-token does not delete or alter existing subscriptions for UAID #1397

Closed
jrconlin opened this issue Jun 3, 2020 · 4 comments · Fixed by #1402
Closed

Ensure that update-token does not delete or alter existing subscriptions for UAID #1397

jrconlin opened this issue Jun 3, 2020 · 4 comments · Fixed by #1402
Assignees
Labels
8 Estimate - xl - Moderately complex, medium effort, some uncertainty.

Comments

@jrconlin
Copy link
Member

jrconlin commented Jun 3, 2020

Getting a report that one of the possible reasons for FxA Send Tab failures happening on android may be that

  1. FxA reports a push publish error
  2. Push on android deletes the native FCM token, and requests a new one.
  3. The new FCM token is sent to the Push Component via PushManager(...).update(...)
  4. UA eventually calls PushManager(...).verify_connection(...) which results in the server returning no subscriptions for the user.

We should ensure that UAID is not altered and the subscriptions are not dropped on a FCM token update. We may also need to file an additional bug against the Android Components Push Module if we think the problem might be in the hand-off.

@jonalmeida
Copy link

This issue seems to be closed, but it's unclear to me what the next steps are. It looks like there were some tests linked to it being closed.

@jrconlin
Copy link
Member Author

jrconlin commented Jul 1, 2020

Basically, it's adding proof that the server is not altering the route data whenever a new vendor token is sent. This ensures that the problem is not with the update path.

In essence, it's reducing the number of unknowns.

@jonalmeida
Copy link

That sounds fair. Should I file an issue in app services then as the next step to investigate? We're still seeing the issue for the Android consumers.

@jrconlin
Copy link
Member Author

jrconlin commented Jul 1, 2020

Yes. I'm hoping that fixing the metric tagging issue helps resolve some of this as well, but for now, it's still very much a mystery.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
8 Estimate - xl - Moderately complex, medium effort, some uncertainty.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants