web-platform-tests/wpt

WebSocketDaemon attempted spawn even when ws_doc_root is null

Open

#8.675 geöffnet am 13. Dez. 2017

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)HTML (2.898 Forks)batch import
good first issueinfrapriority:backlog

Repository-Metriken

Stars
 (4.581 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 5T 23h) (682 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Überprüfen Sie die Diskussion in den Issue Kommentaren (7 Kommentare), um festzustellen, ob die Maintainer eine Präferenz für die Optionen (a), (b) oder (c) angegeben haben. Falls keine klare Richtung vorliegt, implementieren Sie Option (b) gemäß dem Vorschlag des Autors: Überspringen Sie das Starten von WebSocketDaemon, wenn ws doc root null ist. Ändern Sie den relevanten Code in server.py, um vor dem Starten auf null zu prüfen. Aktualisieren Sie bei Bedarf config.default.json, um einen Standardwert festzulegen oder das Verhalten zu dokumentieren.
Tech Stack
python
Domain
backendtesting
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Python basicswpt configuration
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60