Supervisor/supervisor

Eventlistener protocol "len" header changed in Supervisor 4

Open

#1 243 ouverte le 23 mai 2019

Voir sur GitHub
 (33 commentaires) (0 réactions) (0 assignés)Python (1 233 forks)batch import
eventshelp wanted

Métriques du dépôt

Stars
 (7 941 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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

Guide contributeur

Direction de recherche
Examinez le changement dans Supervisor 4 qui a fait que l'en tête 'len' du protocole de l'écouteur d'événements signale des caractères au lieu d'octets. Passez en revue le chemin de code pertinent dans la gestion de l'écouteur d'événements de supervisor, en particulier l'encodage des données stdin. Déterminez si la correction doit rétablir le signalement des octets ou fournir une rétrocompatibilité.
Stack technique
python
Domaine
backendapi
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
PythonSupervisor
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70