web-platform-tests/wpt

WebSocketDaemon attempted spawn even when ws_doc_root is null

Open

#8.675 aberto em 13 de dez. de 2017

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)HTML (2.898 forks)batch import
good first issueinfrapriority:backlog

Métricas do repositório

Stars
 (4.581 stars)
Métricas de merge de PR
 (Mesclagem média 5d 23h) (682 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Reveja a discussão nos comentários da issue (7 comentários) para ver se os mantenedores indicaram uma preferência entre as opções (a), (b) ou (c). Se não houver uma direção clara, implemente a opção (b) conforme sugerido pelo autor: pule a inicialização do WebSocketDaemon quando ws doc root for null. Modifique o código relevante em server.py para verificar se é null antes de inicializar. Atualize config.default.json, se necessário, para definir um valor padrão ou documentar o comportamento.
Pilha de tecnologia
python
Domain
backendtesting
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Python basicswpt configuration
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60