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

fix(static-push): use doesBucketExistV2 ref: #24698 #27367

Merged
merged 2 commits into from Jan 20, 2024
Merged

Conversation

erickgonzalez
Copy link
Contributor

@erickgonzalez erickgonzalez commented Jan 18, 2024

AWSS3Storage
Use a newer method to validate Bucket since some regions require signature v4, and this method takes care of it.

AWSS3PublisherTest
Amazon framework changed the response when the Access Key is not valid, now it returns an error, so the test is no longer valid since we're sending an invalid Key:

com.amazonaws.services.s3.model.AmazonS3Exception: The AWS Access Key ID you provided is not in our records. (Service: Amazon S3; Status Code: 403; Error Code: InvalidAccessKeyId

Copy link

@erickgonzalez erickgonzalez added this pull request to the merge queue Jan 19, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 20, 2024
@erickgonzalez erickgonzalez added this pull request to the merge queue Jan 20, 2024
@erickgonzalez erickgonzalez linked an issue Jan 20, 2024 that may be closed by this pull request
7 tasks
Merged via the queue into master with commit 52eaaa9 Jan 20, 2024
37 checks passed
@erickgonzalez erickgonzalez deleted the issue-24698 branch January 20, 2024 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot push publish to S3 buckets in the us-east-2 region
3 participants