You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by ol...@apache.org on 2015/01/10 16:39:56 UTC

svn commit: r1650756 - /httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/conn/ssl/SSLConnectionSocketFactory.java

Author: olegk
Date: Sat Jan 10 15:39:55 2015
New Revision: 1650756

URL: http://svn.apache.org/r1650756
Log:
Conciser debug logging of cert peer and issuer alternative names

Modified:
    httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/conn/ssl/SSLConnectionSocketFactory.java

Modified: httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/conn/ssl/SSLConnectionSocketFactory.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/conn/ssl/SSLConnectionSocketFactory.java?rev=1650756&r1=1650755&r2=1650756&view=diff
==============================================================================
--- httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/conn/ssl/SSLConnectionSocketFactory.java (original)
+++ httpcomponents/httpclient/trunk/httpclient/src/main/java/org/apache/http/conn/ssl/SSLConnectionSocketFactory.java Sat Jan 10 15:39:55 2015
@@ -432,22 +432,26 @@ public class SSLConnectionSocketFactory
                     this.log.debug(" peer principal: " + peer.toString());
                     final Collection<List<?>> altNames1 = x509.getSubjectAlternativeNames();
                     if (altNames1 != null) {
+                        final List<String> altNames = new ArrayList<String>();
                         for (final List<?> aC : altNames1) {
                             if (!aC.isEmpty()) {
-                                this.log.debug(" peer alternative name: " + aC.get(1));
+                                altNames.add((String) aC.get(1));
                             }
                         }
+                        this.log.debug(" peer alternative names: " + altNames);
                     }
 
                     final X500Principal issuer = x509.getIssuerX500Principal();
                     this.log.debug(" issuer principal: " + issuer.toString());
                     final Collection<List<?>> altNames2 = x509.getIssuerAlternativeNames();
                     if (altNames2 != null) {
+                        final List<String> altNames = new ArrayList<String>();
                         for (final List<?> aC : altNames2) {
                             if (!aC.isEmpty()) {
-                                this.log.debug(" issuer alternative name: " + aC.get(1));
+                                altNames.add((String) aC.get(1));
                             }
                         }
+                        this.log.debug(" issuer alternative names: " + altNames);
                     }
                 } catch (Exception ignore) {
                 }