L'abonnement aux notifications push dans une PWA iOS échoue silencieusement car le SW ne contrôle pas l'application

Contribuer Bug
, ,
1

Bonjour :waving_hand:

Nous avons rencontré un comportement étrange (ou probablement un bug…) lors de la configuration des notifications push dans la PWA iOS. L’interface utilisateur agit comme si l’abonnement aux notifications push était configuré, alors qu’en arrière-plan, rien ne s’est encore produit et l’utilisateur ne recevra jamais de notification push.

(tl;dr : je pense savoir comment le corriger et je peux fournir une PR)

Étapes pour reproduire le problème :footprints:

Le problème concerne l’activation des notifications push immédiatement après l’installation de la PWA sur iOS.

  1. Ouvrez votre instance Discourse dans Safari mobile sur iOS
  2. Ouvrez le menu de partage et choisissez « Ajouter à l’écran d’accueil »
  3. Ouvrez la nouvelle PWA depuis votre écran d’accueil
  4. Vous voyez une bannière vous demandant si vous souhaitez activer les notifications push
  5. Cliquez sur le lien « activer les notifications »
    1. Une boîte de dialogue système apparaît : « xyz souhaite vous envoyer des notifications » — acceptez-la.
  6. En naviguant vers vos préférences de notification, vous verrez que les notifications en direct ne sont pas activées.
    1. En vérifiant côté serveur, vous ne trouverez pas non plus de nouvel enregistrement PushSubscription pour l’utilisateur.

Comportement attendu :books:

Après l’étape 5, vous devriez déjà recevoir une notification indiquant que les notifications sont activées. Il devrait également y avoir un nouvel enregistrement PushSubscription pour votre utilisateur actuel dans la base de données, et son paramètre pour recevoir des notifications en direct dans la PWA devrait être activé.

Problème :bug:

Sur une PWA fraîchement lancée, le service worker est installé et activé, mais ne contrôle pas encore la page. Par conséquent, la vérification dans lib/push-notifications.js, isPushNotificationsSupported(), retournera toujours false, car navigator.serviceWorker.controller renvoie null. Voici les deux lignes importantes :

navigator.serviceWorker.controller &&
navigator.serviceWorker.controller.state === "activated"

Ainsi, l’interface utilisateur suggère à l’utilisateur que tout est en ordre avec sa configuration, mais il ne recevra jamais de notification push.

Au premier chargement, la PWA n’est pas contrôlée et l’appel à subscribe() n’est jamais effectué, même si l’utilisateur a accordé l’autorisation… :bug:

Les utilisateurs peuvent résoudre le problème eux-mêmes s’ils savent comment faire :

  1. Fermer complètement la PWA (glissez-la vers le haut pour la quitter…)
  2. Rouvrez la PWA
  3. Accédez à vos préférences de notification
  4. Réactivez les notifications en direct
  5. → Cette fois, un nouvel enregistrement PushSubscription devrait être créé et les notifications seront configurées.

Correction potentielle :adhesive_bandage:

J’ai contourné ce problème en enregistrant un autre service worker qui appelle skipWaiting() et clients.claim() lors de l’installation/activation. Cela lui permet de prendre immédiatement le contrôle de la PWA, l’abonnement se déroule correctement et l’enregistrement PushSubscription correspondant est créé.

J’ai déployé cela en tant que « correctif urgent » dans l’un de mes plugins personnalisés, afin de pouvoir résoudre la situation pour mes utilisateurs sans avoir à forker le cœur de Discourse…

Il ne s’agit que de deux modifications :

# plugin.rb
register_service_worker "push-notification-setup.js"

// assets/push-notification-setup.js
self.addEventListener("install", function () {
  self.skipWaiting();
});
		
		
self.addEventListener("activate", function (event) {	
  event.waitUntil(self.clients.claim());
});

Je pense que cette correction devrait être intégrée dans le cœur de Discourse. Avec la solution actuelle, le plugin prend un contrôle inutile sur la PWA. Je peux fournir une PR qui ajuste les vérifications dans le cœur de Discourse afin que l’abonnement fonctionne réellement. J’aimerais bien voir cela intégré, mais n’hésitez pas à exprimer vos opinions sur le sujet dès maintenant.

1 « J'aime »
2

Voici la PR contenant une correction pour le cœur de Discourse → FIX: Allow push notification subscriptions on first launch of iOS PWA · Pull Request #39885 · discourse/discourse

Elle résout le problème décrit ici, où l’abonnement échoue silencieusement sans atteindre le serveur. Cependant, ce qui manque encore pour être absolument sûr, c’est que la première notification push reçue ne sera pas routable dans l’application sans le service worker la contrôlant, donc sans que l’utilisateur redémarre l’application.

Je pourrais également ajouter cela à ma PR. Mais je ne suis pas certain à 100 % que cette correction soit celle souhaitée.

1 « J'aime »