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

Validate its a recognised CA #146

Merged
merged 2 commits into from
Jul 18, 2021
Merged

Conversation

lukehinds
Copy link
Member

I had a typo in fuclio serve -ca flag and spent hours trying to work
out why the CA backend was failing. The mispelt flag was being
captured in the middleware handler instead (ca.go line 65) with the
genericCAError from errors.go.

This will capture it before the server even starts and make it a
lot easier to see it's just a simple typo.

Signed-off-by: Luke Hinds [email protected]

Luke Hinds added 2 commits July 18, 2021 21:10
I had a typo in fuclio serve -ca flag and spent hours trying to work
out why the CA backend was failing. The mispelt flag was being
captured in the middleware handler instead (ca.go line 65).

This will capture it before the server even starts and make it a
lot easier to see its just a simple typo.

Signed-off-by: Luke Hinds <[email protected]>
Signed-off-by: Luke Hinds <[email protected]>
@lukehinds lukehinds merged commit 8b1ad5e into sigstore:main Jul 18, 2021
@lukehinds lukehinds deleted the ca-validate branch July 18, 2021 20:39
@cpanato cpanato added this to the 0.2.0 milestone Jul 19, 2021
@cpanato cpanato modified the milestones: 0.2.0, 0.1.1 Jul 26, 2021
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.

None yet

3 participants