web-platform-tests/wpt

WebSocketDaemon attempted spawn even when ws_doc_root is null

Open

#8675 aperta il 13 dic 2017

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)HTML (2898 fork)batch import
good first issueinfrapriority:backlog

Metriche repository

Star
 (4581 star)
Metriche merge PR
 (Merge medio 5g 23h) (682 PR mergiate in 30 g)

Descrizione

If ws_doc_root is null (default in config.default.json), server.py still attempts to spawn WebSocketDaemons, which then fail due to lack of config.

I'm pretty new to wpt so not sure if desired behavior in this case is to

a) Require that ws_doc_root be set b) If ws_doc_root is not set, skip spawning WebSocketDaemon c) Make WebSocketDaemon handle null and default config better

For now, I assuming b) but defer to experts in terms of how this should be handled.

Guida contributor

Direzione di ricerca
Rivedere la discussione nei commenti dell'issue (7 commenti) per verificare se i manutentori hanno indicato una preferenza tra le opzioni (a), (b) o (c). Se non c'è una direzione chiara, implementare l'opzione (b) come suggerito dall'autore: saltare l'avvio di WebSocketDaemon quando ws doc root è null. Modificare il codice pertinente in server.py per controllare se è null prima dell'avvio. Aggiornare config.default.json se necessario per impostare un valore predefinito o documentare il comportamento.
Tech stack
python
Dominio
backendtesting
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Python basicswpt configuration
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60