-
Notifications
You must be signed in to change notification settings - Fork 4.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mobile: pulling server ssl context out of mobile build #34144
Conversation
fe87431
to
d054cde
Compare
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
d054cde
to
c341ec4
Compare
4a88d2b
to
1320e8f
Compare
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/wait-any
source/common/tls/utility.cc
Outdated
@@ -192,22 +192,20 @@ std::vector<std::string> Utility::getSubjectAltNames(X509& cert, int type, bool | |||
|
|||
std::string Utility::generalNameAsString(const GENERAL_NAME* general_name) { | |||
std::string san; | |||
ASN1_STRING* str; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes in this file seem unrelated. Revert? Or if you like them, ASN1_STRING* str = nullptr;
to avoid possibility of uninitialized pointer in future changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah this was a coverage improvement to make up for not e2e testing the factory creation failing. I could alternately add a unit test and not link that in but seems overkill.
/retest |
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
/wait on CI |
/wait on CI |
Risk Level: low
Testing: updated tests
Docs Changes: n/a
Release Notes: n/a
envoyproxy/envoy-mobile#176