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

Certificates are recreated if they share the same common name #27

Closed
nsimons opened this issue Dec 15, 2022 · 0 comments · Fixed by #28
Closed

Certificates are recreated if they share the same common name #27

nsimons opened this issue Dec 15, 2022 · 0 comments · Fixed by #28

Comments

@nsimons
Copy link
Contributor

nsimons commented Dec 15, 2022

To reproduce, execute certyaml twice with the below config:

subject: cn=root
ca: true
---
subject: cn=server
issuer: cn=root
---
subject: cn=server
issuer: cn=root
filename: server-refreshed

Output will be:

→  certyaml -d certs
Loading manifest file: certs.yaml
Reading certificate state file: certs/certs.state
Writing: certs/root.pem certs/root-key.pem
Writing: certs/server.pem certs/server-key.pem
Writing: certs/server-refreshed.pem certs/server-refreshed-key.pem
Writing state: certs/certs.state
→  certyaml -d certs
Loading manifest file: certs.yaml
Reading certificate state file: certs/certs.state
No changes: skipping root
Writing: certs/server.pem certs/server-key.pem
Writing: certs/server-refreshed.pem certs/server-refreshed-key.pem
Writing state: certs/certs.state

Expected:

→  certyaml -d certs
Loading manifest file: certs.yaml
Reading certificate state file: certs/certs.state
Writing: certs/root.pem certs/root-key.pem
Writing: certs/server.pem certs/server-key.pem
Writing: certs/server-refreshed.pem certs/server-refreshed-key.pem
Writing state: certs/certs.state
→  ./certyaml -d certs
Loading manifest file: certs.yaml
Reading certificate state file: certs/certs.state
No changes: skipping root
No changes: skipping server
No changes: skipping server-refreshed
Writing state: certs/certs.state
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 a pull request may close this issue.

1 participant