Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v17] fix: access graph fails to fetch s3 bucket details #50763

Merged
merged 3 commits into from
Jan 6, 2025

Conversation

tigrato
Copy link
Contributor

@tigrato tigrato commented Jan 6, 2025

Backport #50758 to branch/v17

changelog: Fixes a bug where S3 bucket details fail to fetch due to incorrect bucket region.

With SDK V1, AWS get bucket details such as policies, acls, status...
didn't require the specification of the target s3 bucket location. With
the recent changes to support newer versions of the AWS SDK, the get
bucket details started to fail with the following error:

```
BucketRegionError: incorrect region, the bucket is not in 'ap-south-1' region at endpoint '', bucket is in 'eu-central-1' region
status code: 301, request id: QS5C24H12ZV3VNM4, host id: mzVDk4010MPTCFdxyE/XwERX9W35MSge85PG+h5Jvwyvi7MhxdXLaysb2PTZCMY9r1ngBi6Gv6g=, failed to fetch bucket "cyz" acls polic
```

This PR uses `HeadBucket` to retrieve the location of the s3 bucket and
then uses the bucket location client to retrieve the s3 bucket details.

Fixes #50757
Copy link

github-actions bot commented Jan 6, 2025

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
bot/backport-50758-branch/v17 HEAD 1 ✅SUCCEED bot-backport-50758-branch-v17 2025-01-06 13:30:05

@tigrato tigrato added this pull request to the merge queue Jan 6, 2025
Merged via the queue into branch/v17 with commit 7131032 Jan 6, 2025
44 checks passed
@tigrato tigrato deleted the bot/backport-50758-branch/v17 branch January 6, 2025 16:31
@doggydogworld doggydogworld mentioned this pull request Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants