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

feat(S3_in_w_x_flags): Support S3 URIs for custom checks paths and whitelist files. #1090

Conversation

MrCloudSec
Copy link
Member

Context

This new feature will allow users to input S3 URIs when working with whitelists (-w flag) or custom checks paths (-x flag).

Description

If an user input a S3 URI in flags -w or -x, Prowler will automatically read the files from the indicated S3 bucket.
For example:
./prowler -c extra764 -p <profile_name> -w s3://bucket/whitelist.txt
./prowler -c extra764 -p <profile_name> -x s3://bucket/checks

License

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

@MrCloudSec MrCloudSec requested review from a team, toniblyx, jfagoagas and n4ch04 March 30, 2022 12:20
Co-authored-by: Toni de la Fuente <[email protected]>
@MrCloudSec MrCloudSec force-pushed the PRWLR-317-allow-prowler-to-get-whitelist-files-and-custom-checks-from-s-3 branch from a8f91a4 to 031049b Compare March 31, 2022 12:15
@MrCloudSec MrCloudSec requested a review from toniblyx March 31, 2022 12:17
Copy link
Member

@jfagoagas jfagoagas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! Please check my comments above.

As a general comment, review variable braces {}

Co-authored-by: Toni de la Fuente <[email protected]>
@MrCloudSec MrCloudSec force-pushed the PRWLR-317-allow-prowler-to-get-whitelist-files-and-custom-checks-from-s-3 branch from 031049b to a0f5b1f Compare March 31, 2022 13:27
@MrCloudSec MrCloudSec requested a review from jfagoagas March 31, 2022 13:27
@jfagoagas
Copy link
Member

Do we need to include required AWS S3 API permissions?

@MrCloudSec
Copy link
Member Author

Do we need to include required AWS S3 API permissions?

Review the last commit with those changes pls.

Copy link
Member

@jfagoagas jfagoagas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉

New features!! 👏

Copy link
Member

@toniblyx toniblyx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@toniblyx toniblyx merged commit 28fff10 into master Apr 7, 2022
@toniblyx toniblyx deleted the PRWLR-317-allow-prowler-to-get-whitelist-files-and-custom-checks-from-s-3 branch April 7, 2022 18:37
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.

3 participants