851250 - Acknowledge push notification only when app has successfully started and been delivered the notification

Status: RESOLVED INCOMPLETE

(Core :: DOM: Push Subscriptions, defect)

Description

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

It is possible that the app may not get the system message because there isn't enough memory to start the application. The push service should not send an acknowledgement to the server in that case.

Comment 1

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Another case:
App X registers for push
App X is updated, and manifest no longer declares push permission
Push notification comes in.

In this case system messages will not wake up the application. PushService should receive some sort of error from system message and unregister or similar.

Comment 2

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

This isn't urgent at all, but something to think about in the future. The delivery of events to Service Workers has only recently been specced out a bit more - https://github.com/slightlyoff/ServiceWorker/issues/649

Comment 3

[Lina Butler [:lina]]

End-to-end acks have been discussed for Web Push a few times, but I think they're abandoned. We currently nack worker delivery failures; this could be extended to send an end-to-end ack if there's interest.