envoyproxy/envoy

SSL Connection: combine SSL_set_ex_data and SSL_set_app_data

Open

#9867 aperta il 29 gen 2020

Vedi su GitHub
 (2 commenti) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
area/tlshelp wantedtech debt

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Questo problema è una refactoring per combinare due meccanismi di memorizzazione dei dati SSL (SSL set ex data e SSL set app data) in un singolo SSL set app data. Il contesto è la PR #9172 che ha aggiunto ex data extra. L'obiettivo è semplificare tramite consolidamento. Esamina la PR #9172 menzionata e l'uso attuale di entrambe le funzioni nelle classi SslSocket e SslSocketInfo. La ricerca dovrebbe identificare tutti i siti di chiamata e come unirli senza interrompere la funzionalità. Controlla anche i commenti dei manutentori per i criteri di accettazione.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40