cleanup: refine subscriptionIDRegex

andyzhangx · k8s-infra-cherrypick-robot · commit 449fc6d3ad16 · 2026-02-25T12:44:25.000Z
diff --git a/pkg/azurefile/utils.go b/pkg/azurefile/utils.go
@@ -42,6 +42,8 @@ const (
 	tagKeyValueDelimiter = "="
 )
 
+var subscriptionIDRegex = 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}$`)
+
 // lockMap used to lock on entries
 type lockMap struct {
 	sync.Mutex
@@ -361,7 +363,7 @@ func getFileServiceURL(accountName, storageEndpointSuffix string) string {
 }
 
 func isValidSubscriptionID(subsID string) bool {
-	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)
+	return subscriptionIDRegex.MatchString(subsID)
 }
 
 // RemoveOptionIfExists removes the given option from the list of options

Original file line number	Diff line number	Diff line change
`@@ -42,6 +42,8 @@ const (`
`42`	`42`	`tagKeyValueDelimiter = "="`
`43`	`43`	`)`
`44`	`44`
	`45`	+var subscriptionIDRegex = 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}$`)
	`46`	`+`
`45`	`47`	`// lockMap used to lock on entries`
`46`	`48`	`type lockMap struct {`
`47`	`49`	`sync.Mutex`
`@@ -361,7 +363,7 @@ func getFileServiceURL(accountName, storageEndpointSuffix string) string {`
`361`	`363`	`}`
`362`	`364`
`363`	`365`	`func isValidSubscriptionID(subsID string) bool {`
`364`		- 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)
	`366`	`+ return subscriptionIDRegex.MatchString(subsID)`
`365`	`367`	`}`
`366`	`368`
`367`	`369`	`// RemoveOptionIfExists removes the given option from the list of options`