Supervisor/supervisor

Eventlistener protocol "len" header changed in Supervisor 4

Open

#1.243 aberto em 23 de mai. de 2019

Ver no GitHub
 (33 comments) (0 reactions) (0 assignees)Python (1.233 forks)batch import
eventshelp wanted

Métricas do repositório

Stars
 (7.941 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

@plockaby wrote comments of issue #1235:

Migrating from version 3 to version 4, my event listeners read from supervisor over stdin using a byte stream and then did the conversion to utf8 and chose to "backslashreplace" undecodable utf8 data. When I went to version 4 I expected to continue to get a byte stream that I could convert myself to whatever encoding I knew the data to be in. It was really unexpected to discover that supervisor was doing the conversion for me and then telling me the number of characters to read off of stdin and not the number of bytes.

cc: @vsajip

Guia do colaborador

Direção de pesquisa
Investigue a mudança no Supervisor 4 que fez o cabeçalho 'len' do protocolo do ouvinte de eventos relatar caracteres em vez de bytes. Revise o caminho de código relevante no tratamento do ouvinte de eventos do supervisor, especialmente a codificação dos dados stdin. Determine se a correção deve reverter para relatar bytes ou fornecer compatibilidade retroativa.
Pilha de tecnologia
python
Domain
backendapi
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
PythonSupervisor
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70