-
Notifications
You must be signed in to change notification settings - Fork 2.6k
OCSP stapling support #1820
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
OCSP stapling support #1820
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1820 +/- ##
==========================================
- Coverage 94.32% 93.53% -0.79%
==========================================
Files 75 76 +1
Lines 15783 15962 +179
==========================================
+ Hits 14887 14930 +43
- Misses 896 1032 +136
Continue to review full report at Codecov.
|
@chayim Am I reading this wrong, or is this PR about OCSP validation support rather than OCSP stapling? I don't see anywhere the Certificate Status TLS extension here, which is what OCSP stapling is all about. |
This support OCSP validation within the tunnel. The client doesn't currently handle direct stapling. That's a next step. |
Pull Request check-list
Please make sure to review and check all of these items:
$ tox
pass with this change (including linting)?NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.
Description of change
Support for OCSP verification