-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bug: Fix GCM handling #309
Conversation
Closes: #308
We can't fix them, and they could overflow our quota. DEPLOY NOTE: Remove the sentry.io filter for "Bridge authentication error".
@@ -69,7 +69,7 @@ async fn validate_webpush_user( | |||
} | |||
|
|||
/// Drop a user and increment associated metric | |||
async fn drop_user(uaid: Uuid, ddb: &dyn DbClient, metrics: &StatsdClient) -> ApiResult<()> { | |||
pub async fn drop_user(uaid: Uuid, ddb: &dyn DbClient, metrics: &StatsdClient) -> ApiResult<()> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Out of curiosity, why is pub
added here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was needed on an earlier version of this PR, the pub
can be removed now (though it's not terribly important)
@@ -69,7 +69,7 @@ async fn validate_webpush_user( | |||
} | |||
|
|||
/// Drop a user and increment associated metric | |||
async fn drop_user(uaid: Uuid, ddb: &dyn DbClient, metrics: &StatsdClient) -> ApiResult<()> { | |||
pub async fn drop_user(uaid: Uuid, ddb: &dyn DbClient, metrics: &StatsdClient) -> ApiResult<()> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was needed on an earlier version of this PR, the pub
can be removed now (though it's not terribly important)
This patch improves GCM handling, using recently discovered docs of the discontinued protocol.
POST DEPLOY NOTE:
There is currently an ignore in place for
Bridge authentication error
that will need to be removed.Closes: #308