-
Notifications
You must be signed in to change notification settings - Fork 141
Impl SSL_client_hello_get1_extensions_present
and friends
#2561
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
Impl SSL_client_hello_get1_extensions_present
and friends
#2561
Conversation
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.
clang-tidy made some suggestions
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2561 +/- ##
==========================================
- Coverage 78.70% 78.69% -0.01%
==========================================
Files 645 645
Lines 110882 111023 +141
Branches 15671 15685 +14
==========================================
+ Hits 87265 87375 +110
- Misses 22929 22958 +29
- Partials 688 690 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
3cde580
to
0ebf371
Compare
SSL_client_hello_get1_extensions_present
and SSL_client_hello_get_extension_order
SSL_client_hello_get1_extensions_present
and friends
4326849
to
9d485c1
Compare
9d485c1
to
4e95542
Compare
4e95542
to
efb1554
Compare
efb1554
to
1282573
Compare
1282573
to
9363370
Compare
9363370
to
a6c2375
Compare
Description of changes:
Implementation and documentation for:
SSL_client_hello_get1_extensions_present
SSL_client_hello_get_extension_order
SSL_client_hello_get0_legacy_version
Testing:
Added new tests for coverage.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.