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.

Sign up for GitHub

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

feat: use list_buckets check bucket exists #212

Merged

evoxmusic merged 1 commit into Qovery:main from Cluas:feat-use_list_buckets_check_bucket

Aug 14, 2022

Contributor

Cluas commented Aug 12, 2022 •

edited

Loading

get_bucket_accelerate_configuration will return MethodNotAllowed (anyway, I encountered this error), so exists check always failed.

change to list_buckets and iterate to check if the bucket name already exists


feat: use list_buckets instead of get_bucket_accelerate_configuration

e80dbc4

Contributor

evoxmusic commented Aug 14, 2022

Thank you for the fix :)

evoxmusic approved these changes

View reviewed changes

evoxmusic merged commit b1ec9be into Qovery:main

gugacavalieri mentioned this pull request

AWS Cross Account S3 Buckets #233

Open

gugacavalieri mentioned this pull request

Fix: #233 | Use get_bucket_location to check if S3 Bucket exists #240

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment