Wanted to extend hpack encoders or allow different Encoder and Decoders to be used.
Maintainer antworten meist innerhalb von 4 Tagen
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- python
- Bereich
- api, networking
Rechercherichtung
Lies zunächst h2.connection.H2Connection und H2Configuration, und vergleiche dann, wie der aktuelle Encoder und Decoder initialisiert werden, mit der im Issue gezeigten Umgehungslösung über eine Subklasse. Die Arbeit ist abgeschlossen, wenn Aufrufer alternative Encoder- und Decoder-Implementierungen verwenden können, ohne eine eigene H2Connection-Subklasse pflegen zu müssen, und der Konfigurationspfad abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
I have a library coming out which I made as a drop in replacement for python hpack called cyhpack and I wanted to try developing a better solution for injecting my own Decoder and Encoder which runs ls-hpack under the hood. As of currently, here is my workaround but I was wondering if a better implementation could be implemented such as a factory attribute or an Abstract class could be made to accommodate my version and the pure python one as well?
from __future__ import annotations
from h2.config import H2Configuration
from h2.connection import H2Connection as _H2Connection
from .hpack import Decoder, Encoder
class H2Connection(_H2Connection):
"""
Contains an overridable version of H2Connection that drops
python hpack in exchange for cyhpack for faster performance
and speed.
"""
def __init__(self, config: H2Configuration | None = None):
super().__init__(config)
self.encoder = Encoder()
self.decoder = Decoder()
- Vorherrschende Sprache
- Python
- Sterne
- 1k
- Forks
- 197
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus python-hyper/h2
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
python-hyper/h2#1330 ·
Maintainer antworten meist innerhalb von 4 Tagen
-
content-length is not policed when a stream ends with a trailers sectionEvtl. vergeben @feiiiiii5 hat das vor 30 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 76/100
python-hyper/h2#1328 ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
python-hyper/h2#1280 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
python-hyper/h2#1239 · 3 Reaktionen ·
Maintainer antworten meist innerhalb von 4 Tagen
-
ProtocolError on receive_data after connection is closedEvtl. vergeben @adarshx01 hat das vor 69 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 42/100
python-hyper/h2#1199 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 4 Tagen
Alle Issues in python-hyper/h2
Ähnliche Issues
-
request-theme
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 66/100
LizardByte/ThemerrDB#9575 ·
Maintainer antworten meist innerhalb von 1 Tag
-
author-page correction
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
acl-org/acl-anthology#10599 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
zilliztech/memsearch#818 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[FAQ] Commit ID for ProjectsEvtl. vergeben @github-actions hat das heute übernommen. Offenfaq-proposal
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 84/100
DataTalksClub/faq#503 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 2 Tagen
-
README.zh-CN/ja/es still hard-code 501 test counts after README.md went count-freeEvtl. vergeben @Aaqibhafeezkhan hat das heute übernommen. Offenbeginner documentation good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
BistaDinesh03/critique#164 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag