python-hyper / python-hyper/h2
The connection should emit a WindowUpdated event on INITIAL_WINDOW_SIZE change
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 1k
- Forks
- 187
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
I think the connection should emit a WindowUpdated event on receipt of a settings frame that changes the INITIAL_WINDOW_SIZE. This is because this frame is valid after the receipt of headers and hence it implicitly updates the window (see _flow_control_change_from_settings). Without emitting this frame implementations have to specifically look for this settings change and respond appropriately themselves.
(I'll implement this, just wanted to get views on whether this is the correct thing to do).
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
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.
Piste de recherche
Commencez dans h2/connection.py, à _flow_control_change_from_settings, que l’issue identifie comme le chemin pertinent du contrôle de flux. Suivez la manière dont le traitement des settings frames expose actuellement les événements de connexion et vérifiez le comportement environnant avant d’effectuer la modification. Le travail est terminé lorsqu’un INITIAL_WINDOW_SIZE modifié entraîne un événement WindowUpdated, tout en préservant le comportement existant pour les autres settings.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- backend-api-design, networking
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100