envoyproxy/envoy

Custom headers are not available to the SNI `override_auto_sni_header` when added using `request_headers_to_add`

Open

#18.308 geöffnet am 29. Sept. 2021

Auf GitHub ansehen
 (3 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/httparea/tlshelp wanted

Repository-Metriken

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

Beschreibung

Title

Custom headers are not available to the SNI override_auto_sni_header when added using request_headers_to_add

Description

We now allow a customer header to populate the SNI for the upstream requests [Link]. But, if the custom header is not coming from downstream and gets added anywhere using request_headers_to_add then it's value is not available here as this piece of code to populate the SNI gets invoked before we add/modify the request headers probably here.

Contributor Guide

Research-Richtung
Untersuchen Sie die Reihenfolge der Operationen in router.cc: Schauen Sie sich speziell die Zeilen 517 und 648 an, um die Reihenfolge von Header Hinzufügung und SNI Befüllung zu verstehen. Stellen Sie fest, ob das Verschieben der SNI Befüllung nach der Header Hinzufügung das Problem behebt.
Tech Stack
cpp
Domain
backend
Issue Type
Fehler
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-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Envoy internalsC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
20