entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 11 Releases Wiki Activity

ajouter du cache sur les entêtes d'authentification VAPID (#70987) #200

Merged
bdauvergne merged 2 commits from wip/70987-pwa-utiliser-correctement-pywebp into main 2024-02-23 11:17:36 +01:00
Conversation 3 Commits 2 Files Changed 4 +123 -52
bdauvergne commented 2023-12-07 12:23:08 +01:00
Owner
Copy Link

Les tests s'exécutent avec la dernière version de py-vapid mais pas pywebpush et http-ece vu que la partie webpush est entièrement mocké (on a pas de serveur push bouchon pour tester).

Les tests s'exécutent avec la dernière version de py-vapid mais pas pywebpush et http-ece vu que la partie webpush est entièrement mocké (on a pas de serveur push bouchon pour tester).
bdauvergne added 1 commit 2023-12-07 12:23:08 +01:00
gitea/combo/pipeline/head There was a failure building this commit Details
94b0e90909 pwa: conserve VAPID headers in cache for 12 hours (#70987) 
The JSON webtoken is valid for 24 hours but only kept for 23 hours, to
prevent any use after expiration.
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from 94b0e90909 to bb5b40f613 2023-12-07 14:14:28 +01:00 Compare
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from bb5b40f613 to b0f998b5c4 2024-01-09 16:59:43 +01:00 Compare
bdauvergne commented 2024-01-09 17:00:11 +01:00
Author
Owner
Copy Link

Rebasé au dessus de #70988 pour avoir le TTL à 30 jours.

Rebasé au dessus de #70988 pour avoir le TTL à 30 jours.
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from b0f998b5c4 to f8886c77c2 2024-01-09 17:21:03 +01:00 Compare
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from f8886c77c2 to 47c86de394 2024-01-09 17:42:08 +01:00 Compare
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from 47c86de394 to 13c81c3d45 2024-01-10 11:06:06 +01:00 Compare
bdauvergne commented 2024-01-10 11:07:25 +01:00
Author
Owner
Copy Link

J'ai ajouté un nouveau commit pour poser "Urgency: low" lors du PUSH, qui d'après la RFC concernée me semble plus correspondre à ce qu'on fait (sans le header la valeur par défaut est normal), voir https://datatracker.ietf.org/doc/html/rfc8030#section-5.3:

   +----------+-----------------------------+--------------------------+
   | Urgency  | Device State                | Example Application      |
   |          |                             | Scenario                 |
   +----------+-----------------------------+--------------------------+
   | very-low | On power and Wi-Fi          | Advertisements           |
   | low      | On either power or Wi-Fi    | Topic updates            |
   | normal   | On neither power nor Wi-Fi  | Chat or Calendar Message |
   | high     | Low battery                 | Incoming phone call or   |
   |          |                             | time-sensitive alert     |
   +----------+-----------------------------+--------------------------+
J'ai ajouté un nouveau commit pour poser "Urgency: low" lors du PUSH, qui d'après la RFC concernée me semble plus correspondre à ce qu'on fait (sans le header la valeur par défaut est normal), voir https://datatracker.ietf.org/doc/html/rfc8030#section-5.3: ``` +----------+-----------------------------+--------------------------+ | Urgency | Device State | Example Application | | | | Scenario | +----------+-----------------------------+--------------------------+ | very-low | On power and Wi-Fi | Advertisements | | low | On either power or Wi-Fi | Topic updates | | normal | On neither power nor Wi-Fi | Chat or Calendar Message | | high | Low battery | Incoming phone call or | | | | time-sensitive alert | +----------+-----------------------------+--------------------------+ ```
csechet approved these changes 2024-01-24 16:53:10 +01:00
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from 13c81c3d45 to 3461e0dc99 2024-02-22 16:15:11 +01:00 Compare
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from 3461e0dc99 to cb3e48f9b1 2024-02-22 17:04:50 +01:00 Compare
bdauvergne commented 2024-02-22 17:06:51 +01:00
Author
Owner
Copy Link

Je suis repassé sur le code ne testant en direct avec tous les tickets afférents appliqués, la gestion du statut 410 n'était pas bonne, send() renvoie une HttpResponse et ne lève pas de WebPushException comme le faisait pywebpush.webpush()

Je suis repassé sur le code ne testant en direct avec tous les tickets afférents appliqués, la gestion du statut 410 n'était pas bonne, send() renvoie une HttpResponse et ne lève pas de WebPushException comme le faisait pywebpush.webpush()
bdauvergne added 1 commit 2024-02-22 21:43:44 +01:00
gitea/combo/pipeline/head This commit looks good Details
bab935eb96 pwa: factorize send_webpush (#70987)
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from bab935eb96 to 83fa81bef2 2024-02-22 22:26:15 +01:00 Compare
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from 83fa81bef2 to 9cb2baf695 2024-02-22 22:27:10 +01:00 Compare
bdauvergne force-pushed wip/70987-pwa-utiliser-correctement-pywebp from 9cb2baf695 to a5f8140d66 2024-02-23 11:13:44 +01:00 Compare
bdauvergne merged commit a5f8140d66 into main 2024-02-23 11:17:36 +01:00
bdauvergne deleted branch wip/70987-pwa-utiliser-correctement-pywebp 2024-02-23 11:17:36 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
csechet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/combo#200
No description provided.
Delete Branch "wip/70987-pwa-utiliser-correctement-pywebp"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?