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

Feature request: Exclude rules via CLI arguments #78

Open
kmcquade opened this issue Jan 13, 2020 · 3 comments
Open

Feature request: Exclude rules via CLI arguments #78

kmcquade opened this issue Jan 13, 2020 · 3 comments

Comments

@kmcquade
Copy link

@kmcquade kmcquade commented Jan 13, 2020

Hi, great project. I'd like to make a request. Right now, to exclude rules, you have to modify the code to "a comment containing tfsec:ignore:<RULE> to the offending line in your templates", per the README. It would be very useful if we could do this via CLI args as well, for assessment purposes.

@destornillador
Copy link

@destornillador destornillador commented Jan 15, 2020

I agree with @kmcquade . It would be also helpful if we can explicitly exclude rules from a tfsec file (let's say .tfsec).

In my use case, I have 50+ Terraform modules re-using one CI Pipeline code. I have some modules that need to have a fully open ingress rule and others don't. So, excluding this rule only in some Terraform Modules using a .tfsec in their repository, would be the best option.

Thank you

@liamg liamg added the enhancement label Jan 24, 2020
@edersonbrilhante
Copy link

@edersonbrilhante edersonbrilhante commented Mar 9, 2020

Also could be added the option to ignore some files or folders.

@richard-senior
Copy link

@richard-senior richard-senior commented May 14, 2020

Yes please... we need to be able to exclude specific rules by argument when using the python pre-commit framework.
Something like :

  • id: terraform_tfsec
    args: ['args=--AWS004']

etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.