-
Notifications
You must be signed in to change notification settings - Fork 332
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
Inconsistent opinion on common name field #1474
Comments
Fair comment to make! I think we should say that the use of commonName is discouraged for end-entity certificates, where the SAN extension is simply better. For CA certs commonName is fine. I'll create a PR to try and capture that 🔨 |
SgtCoDFish
added a commit
to SgtCoDFish/cert-manager-website
that referenced
this issue
Sep 5, 2024
See cert-manager#1474 Signed-off-by: Ashley Davis <[email protected]>
SgtCoDFish
added a commit
to SgtCoDFish/cert-manager-website
that referenced
this issue
Sep 5, 2024
See cert-manager#1474 Signed-off-by: Ashley Davis <[email protected]>
fixed in #1548 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
website/content/docs/usage/certificate.md
Lines 66 to 67 in eab7224
As explored and summarized in caddyserver/caddy#3755 (comment), the deprecation of the CN field has some ambiguity. I don't have an opinion one way or the other, but cert-manager discourages its use and documents usage of it in a "Bootstrapping CA Issuers" example:
website/content/docs/configuration/selfsigned.md
Line 100 in eab7224
If
commonName
were removed from that example, applying it would result in an error:The language in the comment should be made less absolute, or the usage of
commonName
in examples should be replaced with a field that is not deprecated.The text was updated successfully, but these errors were encountered: