internal/resource: fix bucket field in error message

bgilbert · bgilbert · commit d9c20a12b88d · 2022-04-29T03:22:35.000-04:00
u.Host doesn't work for ARN URLs.
diff --git a/internal/resource/url.go b/internal/resource/url.go
@@ -457,7 +457,7 @@ func (f *Fetcher) fetchFromS3(u url.URL, dest s3target, opts FetchOptions) error
 		region, err = s3manager.GetBucketRegion(ctx, sess, bucket, regionHint)
 		if err != nil {
 			if aerr, ok := err.(awserr.Error); ok && aerr.Code() == "NotFound" {
-				return fmt.Errorf("couldn't determine the region for bucket %q: %v", u.Host, err)
+				return fmt.Errorf("couldn't determine the region for bucket %q: %v", bucket, err)
 			}
 			return err
 		}

Original file line number	Diff line number	Diff line change
`@@ -457,7 +457,7 @@ func (f *Fetcher) fetchFromS3(u url.URL, dest s3target, opts FetchOptions) error`
`457`	`457`	`region, err = s3manager.GetBucketRegion(ctx, sess, bucket, regionHint)`
`458`	`458`	`if err != nil {`
`459`	`459`	`if aerr, ok := err.(awserr.Error); ok && aerr.Code() == "NotFound" {`
`460`		`- return fmt.Errorf("couldn't determine the region for bucket %q: %v", u.Host, err)`
	`460`	`+ return fmt.Errorf("couldn't determine the region for bucket %q: %v", bucket, err)`
`461`	`461`	`}`
`462`	`462`	`return err`
`463`	`463`	`}`