-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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(misconf): Add terraformplan support #4342
Conversation
365a502
to
82e2fb4
Compare
ddbaca1
to
6b73519
Compare
6b73519
to
e5befe8
Compare
@knqyf263 - I've updated the tests and rebased. Could you take a look? |
github.com/aquasecurity/defsec v0.88.1 | ||
github.com/aquasecurity/defsec v0.88.2-0.20230516215146-673ff3afe374 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note: update to latest when #4474 is merged
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. We need to resolve a conflict, though.
Fixes: #4341 Signed-off-by: Simar <simar@linux.com>
Signed-off-by: Simar <simar@linux.com>
Signed-off-by: Simar <simar@linux.com>
e5befe8
to
19462de
Compare
Will fix that. Need this PR to be merged first #4474 and then I'll rebase this one. |
I realized its easier to rebase the other one on top of this. I'll merge this and rebase the other. |
* feat(misconf): Add terraformplan support Fixes: aquasecurity#4341 Signed-off-by: Simar <simar@linux.com> * update defsec * fix lint Signed-off-by: Simar <simar@linux.com> * remove debug prints Signed-off-by: Simar <simar@linux.com> * update tests Signed-off-by: Simar <simar@linux.com> --------- Signed-off-by: Simar <simar@linux.com>
Description
Adds support to scan terraformplan files.
Related issues
Related PRs
Checklist