envoyproxy/envoy

Deprecate Http2ProtocolOptions.allow_connect

Open

#10.412 geöffnet am 17. März 2020

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/httphelp wantedtech debt

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

PR #9964 introduced user defined HTTP/2 SETTINGS parameters and disallowed overriding "named" parameters (e.g., hpack_table_size) with these custom parameters. However, an exception had to be made for allow_connect, a field for which presence can't be checked due to the use of a scalar type.

An API change must be introduced to remove the exception and enable uniformity in override handling.

Contributor Guide

Research-Richtung
Untersuche die Verwendung von allow connect in Http2ProtocolOptions und entferne die spezielle Behandlung dieses Feldes, um eine einheitliche Überschreibung benannter Einstellungen zu ermöglichen.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65