Skip to content

Commit 0210570

Browse files
MattMencelclaude
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 11d6e02 commit 0210570

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
@@ -218,7 +218,7 @@ type BackendSecurityPolicyAzureCredentials struct {
218218
//
219219
// +optional
220220
// +kubebuilder:validation:MinLength=1
221-
ClientID string `json:"clientID,omitempty"`
221+
ClientID string `json:"clientID"`
222222

223223
// TenantId is a unique identifier for an Azure Active Directory instance.
224224
//

0 commit comments

Comments
 (0)