Merge pull request #2903 from kubernetes-sigs/allow-subsid-uppercase

fix: allow uppercase subscription ID

Author: andyzhangx

pkg/azurefile/utils.go

@@ -361,7 +361,7 @@ func getFileServiceURL(accountName, storageEndpointSuffix string) string {
 }
 
 func isValidSubscriptionID(subsID string) bool {
-	return regexp.MustCompile(`^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$`).MatchString(subsID)
+	return regexp.MustCompile(`^[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}$`).MatchString(subsID)
 }
 
 // RemoveOptionIfExists removes the given option from the list of options

pkg/azurefile/utils_test.go

@@ -1071,6 +1071,11 @@ func TestIsValidSubscriptionID(t *testing.T) {
 			subsID:   "2025-04-15T11:06:21.0000000Z",
 			expected: false,
 		},
+		{
+			desc:     "valid subscription ID - uppercase letters",
+			subsID:   "C9D2281E-DCD5-4DFD-9A97-0D50377CDF76",
+			expected: true,
+		},
 	}
 
 	for _, test := range tests {