Skip to content

DTSPO-34108: clarify Front Door TLS certificate guidance - #680

Merged
manohar-hmcts merged 1 commit into
mainfrom
DTSPO-34108-front-door-certs
Oct 8, 2026
Merged

manohar-hmcts merged 1 commit into
mainfrom
DTSPO-34108-front-door-certs

Conversation

@manohar-hmcts

@manohar-hmcts manohar-hmcts commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Jira link

See DTSPO-34108

Change description

  • Front Door page: replaced the "TLS certificate has been stored in vault" step with how certificates work now (fixes Re: 'Front Door configuration' #195)
  • Default is an Azure-managed certificate, validated by a DNS record the pipeline adds; a Key Vault certificate is only used when an entry sets ssl_mode = "AzureKeyVault" (per terraform-module-frontdoor frontdoor.tf / data.tf; default ssl_mode is FrontDoor in CFT and SDS)
  • Notes that certificate_name without ssl_mode has no effect, and the hosted_externally case

Testing done

  • Checked against the module and CFT/SDS prod tfvars: 9 of 72 CFT and 2 of 33 SDS prod entries use Key Vault
  • CI build, spelling and link checks on this PR

Security Vulnerability Assessment

CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?

  • Yes
  • No

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@manohar-hmcts
manohar-hmcts requested a review from a team as a code owner October 8, 2026 10:54
@manohar-hmcts
manohar-hmcts merged commit 7c687d0 into main Oct 8, 2026
4 checks passed
@manohar-hmcts
manohar-hmcts deleted the DTSPO-34108-front-door-certs branch October 8, 2026 13:30
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.

Re: 'Front Door configuration'

2 participants