-
Notifications
You must be signed in to change notification settings - Fork 9
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 option to list cpe in reports #121
Comments
I'm re-reading http://jeremylong.github.io/DependencyCheck/general/suppression.html, and see that in addition to the CVE (Common Vulnerabilities and Exposures) and CPE (Common Platform Enumeration), suppressions can also be by GAV (Maven Group, Artifact, Version). And re-reading http://jeremylong.github.io/DependencyCheck/general/thereport.html, I see that CPE Confidence and Evidence Count describe the CPE, which could be considered interesting information. And if we look at a DependencyCheck sample report http://jeremylong.github.io/DependencyCheck/general/SampleReport.html, I find the CWE (Common Weakness Enumeration) also very interesting. An idea: Since DependencyCheck can already generate reports with all this information, we could consider optionally (or always) having it generate its detailed reports instead of clj-watson trying to replicate them. But... since we also experimentally support the github-advisory, maybe generating our own detailed reports is the way we have to go? (I've not looked into what data the |
In #55 I show an example of GAV-based suppression, but I got the impression (from somewhere -- don't remember) that it wasn't really the recommended way to go (although it's certainly the easiest and doesn't require knowing the |
There are many acronyms, and I'm still learning what they mean. I see that you suppressed by the I don't know the pros and cons of one suppression technique over another, but DependencyCheck's own custom suppressions always includes a DependencyCheck's issue form for false positives looks like this: Notice that the |
See #101 -- in order to report a false positive, a user needs the
cpe
, which meansclj-watson
needs to report that.I don't know what this would look like in JSON or EDN (or Sarif?) but maybe this only needs to be an option for
stdout*
output options?The text was updated successfully, but these errors were encountered: