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

Create govulncheck.yml #393

Merged
merged 5 commits into from
Mar 5, 2023
Merged

Conversation

fxamacker
Copy link
Owner

@fxamacker fxamacker commented Mar 4, 2023

Add a GitHub Actions workflow to run govulncheck.
According to govulncheck docs:

"Govulncheck reports known vulnerabilities that affect Go code. 
It uses static analysis of source code or a binary's symbol table 
to narrow down reports to only those that could affect the application.

By default, govulncheck makes requests to the Go vulnerability database
at https://vuln.go.dev. Requests to the vulnerability database contain only
module paths, not code or other properties of your program."

Add a GitHub Actions workflow to run govulncheck.
According to govulncheck docs:

    "Govulncheck reports known vulnerabilities that affect Go code. 
    It uses static analysis of source code or a binary's symbol table 
    to narrow down reports to only those that could affect the application.

    By default, govulncheck makes requests to the Go vulnerability database
    at https://vuln.go.dev. Requests to the vulnerability database contain only
    module paths, not code or other properties of your program."
@fxamacker fxamacker added the CI CI using GitHub Actions Workflow label Mar 4, 2023
Fix typo by removing a "-".
Temporarily add '**.yml' to trigger workflow.
@fxamacker
Copy link
Owner Author

@x448 can you take a look to see if I missed anything? 🙏

Copy link
Contributor

@x448 x448 left a comment

Choose a reason for hiding this comment

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

Nice find. 👍

Not a big fan of downloading and running govulncheck without verifying hash of the downloaded file. Maybe do that once they tag a release or two. 🤷

@fxamacker fxamacker merged commit 65bc18c into master Mar 5, 2023
@fxamacker
Copy link
Owner Author

I forgot to squash so it's messy but I don't want to do a force push to cleanup.

@fxamacker fxamacker deleted the fxamacker/create-govulncheck-yml branch June 18, 2023 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI CI using GitHub Actions Workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants