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(extra7147): Handle unsupported AWS regions for Glacier #1101

Merged
merged 1 commit into from
Apr 11, 2022

Conversation

jfagoagas
Copy link
Member

@jfagoagas jfagoagas commented Apr 11, 2022

Motivation

Right now if extra7147 is executed against ap-southeast-3 an unhandled exception pops up because AWS Glacier is not supported in this region.
Screenshot 2022-04-11 at 10 37 21

Prowler output:
7937c8f6-470f-4cba-a2ac-76c2d1212da1

Description

Handle unsupported AWS regions for Glacier.
Screenshot 2022-04-11 at 10 53 41

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jfagoagas jfagoagas requested review from a team, toniblyx, MrCloudSec and n4ch04 April 11, 2022 08:47
@jfagoagas jfagoagas self-assigned this Apr 11, 2022
@jfagoagas jfagoagas force-pushed the extra-7147-doesnt-handle-unsupported-regions branch from 7fe6762 to 66db4e5 Compare April 11, 2022 08:56
@jfagoagas jfagoagas force-pushed the extra-7147-doesnt-handle-unsupported-regions branch from 66db4e5 to bb0203d Compare April 11, 2022 14:03
@jfagoagas jfagoagas requested a review from toniblyx April 11, 2022 14:04
@jfagoagas jfagoagas force-pushed the extra-7147-doesnt-handle-unsupported-regions branch 2 times, most recently from 831514b to 46ee8f7 Compare April 11, 2022 14:09
@jfagoagas jfagoagas force-pushed the extra-7147-doesnt-handle-unsupported-regions branch from 46ee8f7 to 5fb3522 Compare April 11, 2022 14:10
@toniblyx toniblyx merged commit fc7c932 into master Apr 11, 2022
@toniblyx toniblyx deleted the extra-7147-doesnt-handle-unsupported-regions branch April 11, 2022 14:10
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.

2 participants