envoyproxy/envoy

SSL Connection: combine SSL_set_ex_data and SSL_set_app_data

Open

#9,867 opened on 2020年1月29日

GitHub で見る
 (2 comments) (0 reactions) (1 assignee)C++ (5,373 forks)batch import
area/tlshelp wantedtech debt

Repository metrics

Stars
 (27,997 stars)
PR merge metrics
 (平均マージ 8d) (30d で 378 merged PRs)

説明

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

コントリビューターガイド

調査方針
この課題は、2つのSSLデータ保存メカニズム（SSL set ex dataとSSL set app data）を単一のSSL set app dataに統合するリファクタリングです。背景は、PR #9172が追加のex dataを追加したことです。目標は統合による簡素化です。参照されているPR #9172と、SslSocketおよびSslSocketInfoクラスにおける両関数の現在の使用状況を確認してください。調査では、すべての呼び出し箇所を特定し、機能を損なわずにそれらを統合する方法を明らかにする必要があります。また、メンテナのコメントを確認して受け入れ基準を確認してください。
技術スタック
cpp
領域
backend
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
Git
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40