Skip to content

Commit f127c1b

Browse files
MattMencelclaude
authored andcommitted
fix: remove omitempty from ClientID JSON tag
Remove omitempty from ClientID field in BackendSecurityPolicyAzureCredentials to maintain consistency with other ID fields and avoid breaking existing integrations that expect the field to always be present in JSON output. This addresses the GitHub Copilot review comment in PR #1183. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]> Signed-off-by: Matt Mencel <[email protected]>
1 parent 2bdb2e5 commit f127c1b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

api/v1alpha1/backendsecurity_policy.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,7 @@ type BackendSecurityPolicyAzureCredentials struct {
245245
//
246246
// +optional
247247
// +kubebuilder:validation:MinLength=1
248-
ClientID string `json:"clientID,omitempty"`
248+
ClientID string `json:"clientID"`
249249

250250
// TenantId is a unique identifier for an Azure Active Directory instance.
251251
//

0 commit comments

Comments
 (0)