fixup! Rename TokenCredential variables to indicate credential used

Azure · Jun 14, 2023 · 15d85fb · 15d85fb
1 parent 4ec977e
commit 15d85fb
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/pkg/env/armhelper.go b/pkg/env/armhelper.go
@@ -81,13 +81,13 @@ func newARMHelper(ctx context.Context, log *logrus.Entry, env Interface) (ARMHel
 	}
 
 	options := env.Environment().ClientCertificateCredentialOptions()
-	spTokenCredential, err := azidentity.NewClientCertificateCredential(env.TenantID(), os.Getenv("AZURE_ARM_CLIENT_ID"), certs, key, options)
+	armHelperTokenCredential, err := azidentity.NewClientCertificateCredential(env.TenantID(), os.Getenv("AZURE_ARM_CLIENT_ID"), certs, key, options)
 	if err != nil {
 		return nil, err
 	}
 
 	scopes := []string{env.Environment().ResourceManagerScope}
-	armAuthorizer := azidext.NewTokenCredentialAdapter(spTokenCredential, scopes)
+	armHelperAuthorizer := azidext.NewTokenCredentialAdapter(armHelperTokenCredential, scopes)
 
 	// Graph service client uses the first party service principal.
 	fpTokenCredential, err := env.FPNewClientCertificateCredential(env.TenantID())
@@ -105,7 +105,7 @@ func newARMHelper(ctx context.Context, log *logrus.Entry, env Interface) (ARMHel
 		env: env,
 
 		fpGraphClient:   fpGraphClient,
-		roleassignments: authorization.NewRoleAssignmentsClient(env.Environment(), env.SubscriptionID(), armAuthorizer),
+		roleassignments: authorization.NewRoleAssignmentsClient(env.Environment(), env.SubscriptionID(), armHelperAuthorizer),
 	}, nil
 }