-
Notifications
You must be signed in to change notification settings - Fork 726
client/tls12: remove stale ref to SCT verification #2166
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
Conversation
The TLS 1.3 equivalent of this list of steps was already updated to reflect the removal of SCT verification. This commit does the same for the TLS 1.2 client code, renumbering steps as required.
Benchmark resultsInstruction countsSignificant differencesThere are no significant instruction count differences Other differencesClick to expand
Wall-timeSignificant differencesThere are no significant wall-time differences Other differencesClick to expand
Additional informationCheckout details:
|
Docs-only. Going to merge w/ 1 review. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2166 +/- ##
=======================================
Coverage 94.72% 94.72%
=======================================
Files 102 102
Lines 23753 23753
=======================================
Hits 22499 22499
Misses 1254 1254 ☔ View full report in Codecov by Sentry. |
The TLS 1.3 equivalent of this list of steps was already updated to reflect the removal of SCT verification. This commit does the same for the TLS 1.2 client code, renumbering steps as required.