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

Add govulncheck to ci/cd #138

Merged
merged 2 commits into from
Mar 4, 2025

Conversation

ties
Copy link
Collaborator

@ties ties commented Mar 4, 2025

Add govulncheck, a conservative static analyzer that checks the reachability of vulnerabilities.

This provides an upper bound on the actual vulnerabilities that are applicable, and is lower-noise than generic checks that check if a vulnerable library is used.

closes #139

@ties
Copy link
Collaborator Author

ties commented Mar 4, 2025

Your code is affected by 8 vulnerabilities from 1 module and the Go standard library.
This scan also found 0 vulnerabilities in packages you import and 5
vulnerabilities in modules you require, but your code doesn't appear to call
these vulnerabilities.

Add govulncheck, a conservative static analyzer that checks the
reachability of vulnerabilities.

This provides an upper bound on the actual vulnerabilities that
are applicable, and is lower-noise than generic checks that check
if a vulnerable library is used.
@ties ties force-pushed the feature/add-govulncheck branch from cf1c54e to b52f448 Compare March 4, 2025 17:59
No git describe output -> no version after fpm ...`-v` -> build fails.
@ties ties force-pushed the feature/add-govulncheck branch from 5655dc1 to 45dab03 Compare March 4, 2025 18:12
@ties
Copy link
Collaborator Author

ties commented Mar 4, 2025

I think this is a good safeguard, let's add it.

@randomthingsandstuff randomthingsandstuff merged commit 13a5baf into bgp:master Mar 4, 2025
3 checks passed
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.

Implement go vuln in CI pipeline?
2 participants