New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 666838 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Revoke push subscription when notifications are denied

Project Member Reported by sbirch@chromium.org, Nov 18 2016

Issue description

Right now, if a user disables push notifications those are suppressed at the Android UI level. That effectively gives the app in question the ability to send background push notifications which aren't user visible.

Ideally we would unsubscribe on disable and resubscribe on re-enable.
 

Comment 1 by sbirch@chromium.org, Nov 18 2016

Sorry, to be clear: "if a user disables push notifications [in the Android UI]..."

Comment 2 by peter@chromium.org, Jan 27 2017

Components: Blink>PushAPI

Comment 3 by peter@chromium.org, Feb 14 2017

Cc: -bever...@google.com peter@chromium.org

Comment 4 by peter@chromium.org, Jun 30 2017

Cc: awdf@chromium.org
Labels: -Restrict-View-Google
Anita's (implicitly) adding this with her work on site notification channels for Android O. We'll have to keep that in mind when migrating WebAPKs to channels.

For Android pre-O, there's two things to do:
  - For WebAPKs, revoking notification permission for the app should be analogous to revoking permission for the origin. That's safe and simple.
  - For regular Chrome, we need to decide what to do when revoking notification permission for the app. Dropping *all* push subscriptions is an option, but then we lose our metrics on how many people have disabled notifications on the app-level.

Comment 5 by joh...@chromium.org, Jul 18 2017

Status: Available (was: Untriaged)
It's possible that some people might temporarily turn off notifications for Chrome, with the intention of turning them on again later and still receiving push notifications from their existing subscriptions. Even if that is the case though, we should probably unsubscribe them, then deliver a pushsubscriptionchange event if the user re-enables notifications.

> Dropping *all* push subscriptions is an option, but then we lose our metrics on how many people have disabled notifications on the app-level.

Presumably we could just use metrics that log once per startup instead of once per incoming push message.
Project Member

Comment 6 by sheriffbot@chromium.org, Jul 19

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Available (was: Untriaged)

Sign in to add a comment