envoyproxy/envoy

SSL Connection: combine SSL_set_ex_data and SSL_set_app_data

Open

#9.867 geöffnet am 29. Jan. 2020

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (5.373 Forks)batch import
area/tlshelp wantedtech debt

Repository-Metriken

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

Beschreibung

Title: Combine SSL Connection's SSL_set_ex_data and SSL_set_app_data

Description: PR #9172 introduces additional SSL_set_ex_data to be able to update SslExtendedSocketInfo during SSL validation. This issue is to refactor the new SSL_set_ex_data and existing SSL_set_app_data to combine to a single SSL_set_app_data.

Currently: SSL_set_app_data: The ContextImpl's const Network::TransportSocketOptions* options set by the ContextImpl but only if (options && !options->verifySubjectAltNameListOverride().empty()) when the SslSocket is created SSL_set_ex_data: The SslSocketInfo::SslExtendedSocketInfo member set when the SslSocket is subsequently moved to be owned by the constructed SslSocketInfo

Contributor Guide

Research-Richtung
Dieses Issue ist eine Refaktorierung, um zwei SSL Datenspeichermechanismen (SSL set ex data und SSL set app data) in einen einzigen SSL set app data zu kombinieren. Der Kontext ist PR #9172, der zusätzliche ex data hinzugefügt hat. Das Ziel ist die Vereinfachung durch Konsolidierung. Siehe den referenzierten PR #9172 und die aktuelle Verwendung beider Funktionen in den Klassen SslSocket und SslSocketInfo. Die Recherche sollte alle Aufrufstellen identifizieren und wie sie zusammengeführt werden können, ohne die Funktionalität zu beeinträchtigen. Überprüfen Sie auch die Kommentare der Maintainer auf Akzeptanzkriterien.
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.
Halber Tag
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
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40