Make $supportedContentEncodings publicly available or implement encoding negotiation
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
Direzione di ricerca
Start with Minishlink\WebPush\Subscription's constructor, its content-encoding validation, and the create() method mentioned in the request. First determine whether the project wants a public supported-encoding accessor or internal negotiation, then define how browser-provided encodings should be supplied and how the chosen behavior can be verified.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
This is feature request:
Minishlink\WebPush\Subscription in constructor checks $contentEncoding against local variable. That prevents setting unsupported value, but there is no way for developer to easily find encoding which is supported by both browser and library.
Some developers still use default value 'aesgcm', other pass first supported encoding if browser supports PushManager.supportedContentEncodings. In first case, code will fail if some browser drop support for 'aesgcm' and in second case, it will fail if browser implement and prefer new encoding.
I have two possible solutions for this:
1.) Make it protected/private property with public getter. That way, developer can make encoding negotiation with browser externally.
2.) Make negotiation internally in constructor (which would require changing signature) or in create() method by accepting array of encodings, taking first supported one from array.
P.S. Sorry for closing and reopening, I missclicked submit, so closed while completing text.
- Lingua principale
- PHP
- Stelle
- 1.9k
- Fork
- 312
- Merge medio
- 1g 15h
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di web-push-libs/web-push-php
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
web-push-libs/web-push-php#441 ·
-
Opera push not working?Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
web-push-libs/web-push-php#430 · 1 reazione ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
web-push-libs/web-push-php#429 · 3 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 20/100
web-push-libs/web-push-php#412 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
web-push-libs/web-push-php#411 · 5 commenti ·
Tutte le issue di web-push-libs/web-push-php
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
-
sync-en
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
sync-en
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
Yoast/wordpress-seo#23658 ·
I maintainer di solito rispondono entro 3 giorni
-
fixed
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
firefly-iii/firefly-iii#12934 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno