-
-
Notifications
You must be signed in to change notification settings - Fork 380
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
Replace tfsec with trivy on github workflows and renovate.json #1801
Conversation
because tfsec is no longer updated Issue: OWASP#1764
Good morning @za ! Good to see you back here. It seems there are some errors: can you please check https://github.com/OWASP/wrongsecrets/actions/runs/12352280456/job/34468977401?pr=1801 ? |
Hi @commjoen it's almost midnight here ;) How do I set |
ERROR: ``` curl: (3) nested brace in URL position 66 https://github.com/aquasecurity/trivy/releases/download/v/trivy_{{env.TRIVY_VERSION}}_Linux-64bit.deb ^ ```
Following the filename that we downloaded
Hi @commjoen now all checks have passed 🎉 Can you review the PR? |
Thank you for your PR sir! Hope to have a look next week, |
Thanks sir @commjoen ! |
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! Thank you @za !
Hi @commjoen I was wondering, how we execute |
When you do ‘pre-commit install’ it will install the toolchain and do the checks on change within the scope of your new commit. |
Issue: #1764
What kind of changes does this PR include?
Description
Update to trivy because tfsec is no longer updated
Relations
Closes #1764
Checklist: