Skip to content

Add clientSecretCertificateKeyVaultReference to staticwebapp.config.json schema#5452

Merged
madskristensen merged 1 commit intoSchemaStore:masterfrom
tfendtvep-art:add-swa-certificate-properties
Mar 9, 2026
Merged

Add clientSecretCertificateKeyVaultReference to staticwebapp.config.json schema#5452
madskristensen merged 1 commit intoSchemaStore:masterfrom
tfendtvep-art:add-swa-certificate-properties

Conversation

@tfendtvep-art
Copy link
Contributor

Summary

  • Adds clientSecretCertificateKeyVaultReference and clientSecretCertificateThumbprint properties to the azureActiveDirectory > registration object in the staticwebapp.config.json schema.
  • These properties support certificate-based authentication as an alternative to clientSecretSettingName (client secret).

Context

Microsoft's official documentation lists clientSecretCertificateKeyVaultReference and clientSecretCertificateThumbprint as valid configuration properties for Azure Active Directory custom authentication:

The SWA CLI fetches this schema from SchemaStore at deploy time and rejects valid configurations that use these properties because they are missing from the schema. This blocks deployments that use certificate-based auth instead of client secrets.

Related issue: Azure/static-web-apps-cli#944

Test plan

  • Validate that the updated schema still passes SchemaStore CI
  • Confirm a staticwebapp.config.json using clientSecretCertificateKeyVaultReference and clientSecretCertificateThumbprint is accepted by the schema

🤖 Generated with Claude Code

Add clientSecretCertificateKeyVaultReference and
clientSecretCertificateThumbprint to the azureActiveDirectory registration
object. These properties are needed for certificate-based authentication
as documented at https://learn.microsoft.com/en-us/azure/static-web-apps/authentication-custom

Fixes Azure/static-web-apps-cli#944

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2026

Thanks for the PR!

This section of the codebase is owned by @madskristensen and @hyperupcall - if they write a comment saying "LGTM" then it will be merged.

@madskristensen madskristensen merged commit 2314901 into SchemaStore:master Mar 9, 2026
5 checks passed
@madskristensen
Copy link
Contributor

Thanks

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.

2 participants