Only configure STS region for route53 when we obtain it using IDMS

next-stack · Jan 2, 2025 · 3298a8e · 3298a8e
1 parent 951129c
commit 3298a8e
Showing 1 changed file with 22 additions and 19 deletions.
diff --git a/dnsprovider/pkg/dnsprovider/providers/aws/route53/route53.go b/dnsprovider/pkg/dnsprovider/providers/aws/route53/route53.go
@@ -71,34 +71,37 @@ func newRoute53() (*Interface, error) {
 		region = imdsRegionResp.Region
 	}
 
-	stsCfg, err := awsconfig.LoadDefaultConfig(ctx,
-		awsconfig.WithClientLogMode(aws.LogRetries),
-		awslog.WithAWSLogger(),
-		awsconfig.WithRetryer(func() aws.Retryer {
-			return retry.AddWithMaxAttempts(retry.NewStandard(), 5)
-		}),
-		awsconfig.WithRegion(region),
-	)
-	if err != nil {
-		return nil, fmt.Errorf("failed to load default aws config for STS client: %w", err)
-	}
-
 	awsOptions := []func(*awsconfig.LoadOptions) error{
 		awsconfig.WithClientLogMode(aws.LogRetries),
 		awslog.WithAWSLogger(),
 		awsconfig.WithRetryer(func() aws.Retryer {
 			return retry.AddWithMaxAttempts(retry.NewStandard(), 5)
 		}),
-		awsconfig.WithAssumeRoleCredentialOptions(func(aro *stscreds.AssumeRoleOptions) {
-			// Ensure the STS client has a region configured, if discovered by IMDS
-			aro.Client = sts.NewFromConfig(stsCfg)
-		}),
 	}
 
 	if imdsClient != nil {
-		awsOptions = append(awsOptions, awsconfig.WithEC2IMDSRegion(func(o *awsconfig.UseEC2IMDSRegion) {
-			o.Client = imdsClient
-		}))
+		stsCfg, err := awsconfig.LoadDefaultConfig(ctx,
+			awsconfig.WithClientLogMode(aws.LogRetries),
+			awslog.WithAWSLogger(),
+			awsconfig.WithRetryer(func() aws.Retryer {
+				return retry.AddWithMaxAttempts(retry.NewStandard(), 5)
+			}),
+			awsconfig.WithRegion(region),
+		)
+		if err != nil {
+			return nil, fmt.Errorf("failed to load default aws config for STS client: %w", err)
+		}
+
+		awsOptions = append(
+			awsOptions,
+			awsconfig.WithEC2IMDSRegion(func(o *awsconfig.UseEC2IMDSRegion) {
+				o.Client = imdsClient
+			}),
+			awsconfig.WithAssumeRoleCredentialOptions(func(aro *stscreds.AssumeRoleOptions) {
+				// Ensure the STS client has a region configured, if discovered by IMDS
+				aro.Client = sts.NewFromConfig(stsCfg)
+			}),
+		)
 	}
 
 	cfg, err := awsconfig.LoadDefaultConfig(ctx, awsOptions...)