web-platform-tests/wpt

WebSocketDaemon attempted spawn even when ws_doc_root is null

Open

#8 675 ouverte le 13 déc. 2017

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)HTML (2 898 forks)batch import
good first issueinfrapriority:backlog

Métriques du dépôt

Stars
 (4 581 stars)
Métriques de merge PR
 (Merge moyen 5j 23h) (682 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez la discussion dans les commentaires de l'issue (7 commentaires) pour voir si les mainteneurs ont indiqué une préférence parmi les options (a), (b) ou (c). S'il n'y a pas de direction claire, implémentez l'option (b) comme suggéré par l'auteur : ignorer le lancement de WebSocketDaemon lorsque ws doc root est null. Modifiez le code pertinent dans server.py pour vérifier s'il est null avant de lancer. Mettez à jour config.default.json si nécessaire pour définir une valeur par défaut ou documenter le comportement.
Stack technique
python
Domaine
backendtesting
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Python basicswpt configuration
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60