-
Notifications
You must be signed in to change notification settings - Fork 726
Faster TLS1.3 server resumption #2168
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
Benchmark resultsInstruction countsSignificant differencesClick to expand
Other differencesClick to expand
Wall-timeSignificant differencesClick to expand
Other differencesClick to expand
Additional informationCheckout details:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2168 +/- ##
==========================================
- Coverage 94.72% 94.71% -0.01%
==========================================
Files 102 102
Lines 23753 23748 -5
==========================================
- Hits 22499 22493 -6
- Misses 1254 1255 +1 ☔ View full report in Codecov by Sentry. |
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.
Nice!
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.
Cool!
f09bead
to
3a5f510
Compare
3a5f510
to
bee6836
Compare
By default, we send 4 tickets. Before this change, for each one we recomputed `ResumptionMasterSecret`, then used it, then discarded it (and it is a type that requires zeroisation). Instead, hoist this into its own type, and make it once for each set of tickets.
bee6836
to
e02c24a
Compare
First commit fixes a small bit of wastefulness. Its effect is somewhat tempered by #2167 , but worth having.
Second commit expands the scope of #2120.
Indicative performance results:
8019.6 handshakes/sec
8499.08 handshakes/sec
9666.8 handshakes/sec