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

jsoup:0.0.1.rc19: CVE-2015-6748 #339

Closed
kewilson opened this issue Mar 19, 2019 · 3 comments
Closed

jsoup:0.0.1.rc19: CVE-2015-6748 #339

kewilson opened this issue Mar 19, 2019 · 3 comments

Comments

@kewilson
Copy link

From owasp.dependencycheck gradle plugin:
https://www.owasp.org/index.php/OWASP_Dependency_Check

openhtmltopdf-jsoup-dom-converter-0.0.1-RC19.jar:
ids:(com.openhtmltopdf:openhtmltopdf-jsoup-dom-converter:0.0.1-RC19, cpe:/a:jsoup:

guava-20.0.jar:
ids:(com.google.guava:guava:20.0, cpe:/a:google:guava:20.0) : CVE-2018-10237

@danfickle
Copy link
Owner

Hi @kewilson,

Thanks for the report. However, I couldn’t find any evidence that jsoup depends on guava either now or in the past. At version 1.9.1 it had no non-test dependencies and still doesn’t. Do you have any more details?

Additionally, since this is a one file module, with very few users, I was thinking of deleting it and instead post it as a code sample on the wiki. What do you think?

@kewilson
Copy link
Author

Hi @danfickle .. apologies for the delayed response. If you believe providing that functionality in some other way is best that's your call. I'll try to track down the entry point of guava into the report, I found that odd myself.

danfickle added a commit that referenced this issue Apr 2, 2019
Upgrade Jsoup dependency for last release of this sub-module. Users should use W3CDom class from Jsoup instead.
@danfickle
Copy link
Owner

OK, I've marked the module deprecated (for removal) and changed the integration guide to use JSoup's W3CDom helper class (which does the same thing) instead. I'll do one more release with the module (with the latest JSoup) before removing as I wouldn't like to leave the last listed version of a module having a possible security issue.

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

No branches or pull requests

2 participants