Skip to content

tooling: Fix cve scan script#17993

Merged
mattklein123 merged 3 commits intoenvoyproxy:mainfrom
phlax:tooling-cve-scan-fix
Sep 7, 2021
Merged

tooling: Fix cve scan script#17993
mattklein123 merged 3 commits intoenvoyproxy:mainfrom
phlax:tooling-cve-scan-fix

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Sep 5, 2021

Signed-off-by: Ryan Northey ryan@synca.io

Commit Message: tooling: Fix cve scan script
Additional Description:

this is not a perfect solution, but it should at least get the scanner working again

Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Ryan Northey <ryan@synca.io>
@mattklein123 mattklein123 self-assigned this Sep 6, 2021
@mattklein123
Copy link
Copy Markdown
Member

Cool thanks. Can you uncomment the GH action yaml so it runs again?

/wait

Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Ryan Northey <ryan@synca.io>
Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

@mattklein123 mattklein123 merged commit 8da8e73 into envoyproxy:main Sep 7, 2021
tyxia pushed a commit to tyxia/envoy that referenced this pull request Sep 21, 2021
Signed-off-by: Ryan Northey <ryan@synca.io>

Signed-off-by: phlax <phlax@users.noreply.github.com>
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