Skip to content

Commit aba658f

Browse files
m-nashCopilot
andauthored
Remove duplicate top-level required on credential definition (Azure#58408)
Both Azure.Core and System.ClientModel ConfigurationSchema.json segments declared `required: ["CredentialSource"]` at the top of `definitions/credential`. When the appsettings schema combiner merges segments by definition name, it concatenates `required` arrays without dedup, producing `["CredentialSource", "CredentialSource"]` in the consumer's combined AppSettingsSchema.json. System.ClientModel already requires CredentialSource, so removing it from Azure.Core's top-level credential definition eliminates the duplicate while preserving all `if`-clause requireds and the chainableCredential definition. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 369b035 commit aba658f

1 file changed

Lines changed: 0 additions & 3 deletions

File tree

sdk/core/Azure.Core/schema/ConfigurationSchema.json

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -121,9 +121,6 @@
121121
]
122122
}
123123
},
124-
"required": [
125-
"CredentialSource"
126-
],
127124
"allOf": [
128125
{
129126
"if": {

0 commit comments

Comments
 (0)