python-hyper / python-hyper/h2
The connection should emit a WindowUpdated event on INITIAL_WINDOW_SIZE change
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 1k
- Forks
- 187
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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).
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in h2/connection.py bei _flow_control_change_from_settings, das im Issue als der relevante Flow-Control-Pfad identifiziert wird. Verfolge, wie die Verarbeitung von Settings-Frames derzeit Connection-Events offenlegt, und prüfe das umgebende Verhalten, bevor du die Änderung vornimmst. Als abgeschlossen gilt die Änderung, wenn ein geändertes INITIAL_WINDOW_SIZE zu einem WindowUpdated-Event führt und das bestehende Verhalten für andere Settings erhalten bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- backend-api-design, networking
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100