Skip to content

docs: add SAN extension to secure ingress certificate generation commands#17335

Open
Ahmedaltu wants to merge 2 commits intoistio:masterfrom
Ahmedaltu:fix/secure-ingress-san-certificates
Open

docs: add SAN extension to secure ingress certificate generation commands#17335
Ahmedaltu wants to merge 2 commits intoistio:masterfrom
Ahmedaltu:fix/secure-ingress-san-certificates

Conversation

@Ahmedaltu
Copy link
Copy Markdown

Fixes #11675

What this PR does

Adds the -extfile flag with a Subject Alternative Name (SAN) extension
to all openssl x509 certificate signing commands in the secure ingress
documentation, and adds a note explaining the browser compatibility requirement.

Why

Since Chrome 58, browsers require certificates to have the domain name in
the Subject Alternative Name (SAN) extension, not just the Common Name (CN)
field. The current commands generate certificates without a SAN, which causes:

  • ERR_CERT_COMMON_NAME_INVALID in Chrome
  • Subject Alternative Name Missing error

This issue has been open since August 2022 (#11675).

Changes

  1. Added -extfile <(printf "subjectAltName=DNS:<domain>") to 4 certificate
    signing commands:

    • httpbin.example.com (example_certs1)
    • httpbin.example.com (example_certs2)
    • helloworld.example.com
    • client.example.com
  2. Added a tip note explaining that the certificates are for testing only,
    require SAN for modern browsers, and will not be automatically trusted.

Type of change

  • Documentation fix

@Ahmedaltu Ahmedaltu requested a review from a team as a code owner April 20, 2026 18:26
@istio-testing istio-testing added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. needs-ok-to-test labels Apr 20, 2026
@istio-testing
Copy link
Copy Markdown
Contributor

Hi @Ahmedaltu. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ok-to-test size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Generate client and server certificates and keys

2 participants