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.

Sign up for GitHub

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

Jump to bottom

OKHTTP protocol trust all SSL certificates #615

Closed
jnioche opened this issue Sep 26, 2018 · 2 comments
Closed

OKHTTP protocol trust all SSL certificates #615

jnioche opened this issue Sep 26, 2018 · 2 comments
Labels
core enhancement
Milestone
1.11

Comments

@jnioche
Copy link
Contributor

jnioche commented Sep 26, 2018

This would prevent the issue below

 javax.net.ssl.SSLHandshakeException: sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
javax.net.ssl.SSLHandshakeException: sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
at sun.security.ssl.Alerts.getSSLException(Alerts.java:192)
@jnioche jnioche added this to the 1.11 milestone Sep 26, 2018
@jnioche
Copy link
Contributor Author

jnioche commented Sep 26, 2018

@sebastian-nagel feel free to port to the Nutch implementation if find it useful. I saw that a similar thing was recently added for the other protocol there

@sebastian-nagel
Copy link
Contributor

Thanks, that was already my plan after Markus provided the patch for protocol-http.

jcruzmartini pushed a commit to jcruzmartini/storm-crawler that referenced this issue Dec 3, 2018
@jnioche @jcruzmartini
OKHTTP protocol trust all SSL certificates, fixes apache#615
289344c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core enhancement
Projects
None yet
Milestone
1.11
Development

No branches or pull requests
2 participants
@jnioche @sebastian-nagel