You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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.
The text was updated successfully, but these errors were encountered: