Skip to content

Commit

Permalink
[FIXED JENKINS-12629] Using jenkins-cli connecting to HTTPS port fail…
Browse files Browse the repository at this point in the history
…s due to hostname mismatch in certificate.

- NoCheckTrustManager needed to implement X509TrustManager not just plain TrustManager

(cherry picked from commit 0ab7988)
  • Loading branch information
stephenc authored and olivergondza committed Oct 9, 2013
1 parent f53e3ec commit fb71d62
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cli/src/main/java/hudson/cli/NoCheckTrustManager.java
@@ -1,13 +1,13 @@
package hudson.cli;

import javax.net.ssl.TrustManager;
import javax.net.ssl.X509TrustManager;
import java.security.cert.CertificateException;
import java.security.cert.X509Certificate;

/**
* @author Kohsuke Kawaguchi
*/
public class NoCheckTrustManager implements TrustManager {
public class NoCheckTrustManager implements X509TrustManager {
public void checkClientTrusted(X509Certificate[] x509Certificates, String s) throws CertificateException {
}

Expand Down

0 comments on commit fb71d62

Please sign in to comment.