Skip to content
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

[ansible/artifactory] Enable OCSP stapling support artifactory_nginx_ssl #358

Closed
wants to merge 7 commits into from

Conversation

bbaassssiiee
Copy link
Contributor

@bbaassssiiee bbaassssiiee commented Dec 21, 2023

PR Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Title of the PR starts with installer/product name (e.g. [ansible/artifactory])
  • CHANGELOG.md updated
  • Variables and other changes are documented in the README.md

What this PR does / why we need it:
This implements a feature request.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
#355

Special notes for your reviewer:

@PolaricEntropy
Copy link
Contributor

@chukka Is there anything blocking this?

@vasukinjfrog
Copy link
Collaborator

@PolaricEntropy , @bbaassssiiee - The handshake difference with OCSP stapling, can that verbose output be provided, a few request samples which can show the CA verification is skipped off will also be helpful and would serve as the testing evidence. Thanks

@bbaassssiiee
Copy link
Contributor Author

Lets Encrypt announced the death of OSCP
https://letsencrypt.org/2024/12/05/ending-ocsp/

@bbaassssiiee bbaassssiiee deleted the feature/355 branch January 30, 2025 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants