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

verify_connection appears to be returning an empty list for users with active subscriptions #1442

Closed
jrconlin opened this issue Oct 19, 2020 · 4 comments
Assignees
Labels
5 Estimate - l - Moderately complex, will require some effort but clearly defined.

Comments

@jrconlin
Copy link
Member

See mozilla/application-services#3314 (comment)

Verify that users, particularly mobile users, channels are being registered correctly and are returned as part of the verify_connection() call. This is impeding work and analysis on Send Tab reliability.

@jrconlin jrconlin added the 5 Estimate - l - Moderately complex, will require some effort but clearly defined. label Oct 19, 2020
@rfk
Copy link

rfk commented Oct 21, 2020

Ref mozilla/application-services#3314 (comment), I think that the server may be returning registered channels in cases where the client expects it to not, rather than not returning channels when the client expects that it should.

@jrconlin
Copy link
Member Author

I'm not sure how this might happen, unless either the unsubscribe call failed for a given ChannelID and the client did not handle it properly, or if there might be some race condition where the unsubscribe was still pending and a verify was submitted.

We have a number of both unit and integration tests that check to ensure that a given channel is removed, while other channels remain in tact, during an unsubscribe. Still, chaos is ever present. I'm just not quite sure how to pursue this on the server.

@jrconlin
Copy link
Member Author

De-prioritizing this bug since no work is currently required for the autopush server.

@rfk
Copy link

rfk commented Nov 5, 2020

Based on discussion in the linked appservices issue, I believe the server is behaving as expected here.

@rfk rfk closed this as completed Nov 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
5 Estimate - l - Moderately complex, will require some effort but clearly defined.
Projects
None yet
Development

No branches or pull requests

2 participants