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

x/vuln/cmd/govulncheck: cache results #57150

Open
zpavlinovic opened this issue Dec 7, 2022 · 0 comments
Open

x/vuln/cmd/govulncheck: cache results #57150

zpavlinovic opened this issue Dec 7, 2022 · 0 comments
Assignees
Labels
vulncheck or vulndb Issues for the x/vuln or x/vulndb repo

Comments

@zpavlinovic
Copy link
Contributor

Imagine a user runs govulncheck ./... and discovers some vulnerabilities for a very large project. The user may next want to see the detailed trace by running govulncheck -v ./.... Now govulncheck will do the job from scratch which isn't nice, as this will take some time.

Imagine that an IDE or a wrapper program reports issues by packaging the result of govulncheck -json. The user may want to see a more concise result as govulncheck. Now govulncheck will do the job from scratch again which isn't nice, as this will take some time.

Caching results here would be great. One idea is to cache vulncheck.Result. This should be much less memory intensive and more convenient than caching intermediate call graphs. Caching can be done in between govulncheck calls where db has not been changed as well as the code under analysis. The latter can be possibly implemented by looking how go build cache works.

Thanks @hyangah for suggesting this idea.

@gopherbot gopherbot added the vulncheck or vulndb Issues for the x/vuln or x/vulndb repo label Dec 7, 2022
@gopherbot gopherbot modified the milestones: Unreleased, vuln/unplanned Dec 7, 2022
@zpavlinovic zpavlinovic self-assigned this Dec 7, 2022
@zpavlinovic zpavlinovic added vulncheck or vulndb Issues for the x/vuln or x/vulndb repo and removed vulncheck or vulndb Issues for the x/vuln or x/vulndb repo labels Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vulncheck or vulndb Issues for the x/vuln or x/vulndb repo
Projects
None yet
Development

No branches or pull requests

2 participants