envoyproxy/envoy

SSL Connection: combine SSL_set_ex_data and SSL_set_app_data

Open

#9 867 ouverte le 29 janv. 2020

Voir sur GitHub
 (2 commentaires) (0 réactions) (1 assigné)C++ (5 373 forks)batch import
area/tlshelp wantedtech debt

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Cette issue est une refactorisation visant à combiner deux mécanismes de stockage de données SSL (SSL set ex data et SSL set app data) en un seul SSL set app data. Le contexte est la PR #9172 qui a ajouté des ex data supplémentaires. L'objectif est de simplifier par consolidation. Examinez la PR #9172 référencée et l'utilisation actuelle des deux fonctions dans les classes SslSocket et SslSocketInfo. La recherche doit identifier tous les sites d'appel et comment les fusionner sans casser la fonctionnalité. Vérifiez également les commentaires des mainteneurs pour les critères d'acceptation.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40