Attempt to close with local-only codes (e.g. 1006) results in normal close (1000)
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 38/100
- Type d'issue
- Bug
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- networking
Piste de recherche
Start at src/wsproto/frame_protocol.py lines 576-577 and trace how an attempted close with code 1006 is handled. Confirm the WebSocket protocol's expected behavior for local-only close codes, then define a regression test showing that 1006 is not converted into a normal 1000 closure.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Currently, if you close a websocket with code 1006 (which is not allowed), the result is a 1000-code closure:
https://github.com/python-hyper/wsproto/blob/main/src/wsproto/frame_protocol.py#L576-L577
I suspect this should instead be either a 1002 error or an exception, but I'm not certain.
This resulted in a latent bug in hypercorn (an ASGI web server that uses wsproto) where websockets were closed with code 1000 on internal server errors:
https://github.com/pgjones/hypercorn/pull/112
- Langage dominant
- Python
- Étoiles
- 303
- Forks
- 61
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de python-hyper/wsproto
-
CompressionOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
python-hyper/wsproto#199 · 5 commentaires ·
-
Type compatibility with `h11._headers.Header`Peut-être pris @gyanu2507 l’a pris il y a 58 jours. Ouverte
Difficulté 1/5 1-3 heures Accessibilité débutants 48/100
python-hyper/wsproto#173 · 1 commentaire ·
Toutes les issues de python-hyper/wsproto
Issues similaires
-
link-rot
Difficulté 1/5 Moins d'une heure Accessibilité débutants 67/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
developmentseed/titiler-cmr#265 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Insecure Brotli version <1.2.1Peut-être pris @jamescw19 l’a pris aujourd’hui. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
stac-utils/stac-fastapi-pgstac#438 · 1 réaction ·
-
bug triage
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
Les mainteneurs répondent en général sous 1 jour