Skip to content

Use /crl/pem instead of /crl/der to fetch CRLs - #63

Open
Ptijohn wants to merge 4 commits into
mainfrom
use-crl-pem-instead-of-der
Open

Ptijohn wants to merge 4 commits into
mainfrom
use-crl-pem-instead-of-der

Conversation

@Ptijohn

@Ptijohn Ptijohn commented Jun 30, 2026 •

Copy link
Copy Markdown

As a (hopefully) temporary fix to hashicorp/vault#32018

Won't merge for now, to give some time on the original ticket.

@Ptijohn
Ptijohn requested a review from FaustXVI June 30, 2026 07:54
Comment thread example/actions/revoke-certificate.sh Outdated
@Ptijohn
Ptijohn requested a review from FaustXVI September 23, 2026 09:22
Comment thread src/scripts/orca-protocol/sudoer/export_crls.nix Outdated
@Ptijohn
Ptijohn requested a review from FaustXVI September 23, 2026 09:27

@FaustXVI FaustXVI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still need to test it, but code LGTM

@FaustXVI FaustXVI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After testing :

  • we need to add openssl to the packages
  • we need to do the same for the certificates created at CA creation

@Ptijohn
Ptijohn requested a review from FaustXVI September 23, 2026 10:13

This branch has not been deployed

No deployments
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.

2 participants