From 317e2da00074205a99b7a9a05be41a63271a85a1 Mon Sep 17 00:00:00 2001 From: Taylor Chaparro <33099255+notchairmk@users.noreply.github.com> Date: Fri, 19 Feb 2021 02:16:31 -0800 Subject: [PATCH] azure client sig fix: don't overwrite subscription id if unset (#10659) --- builder/azure/arm/azure_client.go | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/builder/azure/arm/azure_client.go b/builder/azure/arm/azure_client.go index db4244907..26dadc336 100644 --- a/builder/azure/arm/azure_client.go +++ b/builder/azure/arm/azure_client.go @@ -233,7 +233,9 @@ func NewAzureClient(subscriptionID, sigSubscriptionID, resourceGroupName, storag azureClient.GalleryImageVersionsClient.ResponseInspector = byConcatDecorators(byInspecting(maxlen), errorCapture(azureClient)) azureClient.GalleryImageVersionsClient.UserAgent = fmt.Sprintf("%s %s", useragent.String(version.AzurePluginVersion.FormattedVersion()), azureClient.GalleryImageVersionsClient.UserAgent) azureClient.GalleryImageVersionsClient.Client.PollingDuration = sharedGalleryTimeout - azureClient.GalleryImageVersionsClient.SubscriptionID = sigSubscriptionID + if sigSubscriptionID != "" { + azureClient.GalleryImageVersionsClient.SubscriptionID = sigSubscriptionID + } azureClient.GalleryImagesClient = newCompute.NewGalleryImagesClientWithBaseURI(cloud.ResourceManagerEndpoint, subscriptionID) azureClient.GalleryImagesClient.Authorizer = autorest.NewBearerAuthorizer(servicePrincipalToken) @@ -241,7 +243,9 @@ func NewAzureClient(subscriptionID, sigSubscriptionID, resourceGroupName, storag azureClient.GalleryImagesClient.ResponseInspector = byConcatDecorators(byInspecting(maxlen), errorCapture(azureClient)) azureClient.GalleryImagesClient.UserAgent = fmt.Sprintf("%s %s", useragent.String(version.AzurePluginVersion.FormattedVersion()), azureClient.GalleryImagesClient.UserAgent) azureClient.GalleryImagesClient.Client.PollingDuration = pollingDuration - azureClient.GalleryImagesClient.SubscriptionID = sigSubscriptionID + if sigSubscriptionID != "" { + azureClient.GalleryImagesClient.SubscriptionID = sigSubscriptionID + } keyVaultURL, err := url.Parse(cloud.KeyVaultEndpoint) if err != nil {