cxf-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bhar...@apache.org
Subject svn commit: r890056 - in /cxf/trunk: rt/core/src/main/java/org/apache/cxf/endpoint/ rt/transports/http/src/main/java/org/apache/cxf/transport/http/ tools/corba/src/main/java/org/apache/cxf/tools/corba/idlpreprocessor/
Date Sun, 13 Dec 2009 12:16:17 GMT
Author: bharath
Date: Sun Dec 13 12:16:17 2009
New Revision: 890056

URL: http://svn.apache.org/viewvc?rev=890056&view=rev
Log:
@Bug: CXF-2581
@Desc During logging, we invoke toString on an array, which will generate junk result. Fixed
this by invoking Arrays.toString to convert the array into a readable String that gives the
contents of the array.
@Test Units Tests

Modified:
    cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java
    cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
    cxf/trunk/tools/corba/src/main/java/org/apache/cxf/tools/corba/idlpreprocessor/DefaultIncludeResolver.java

Modified: cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java?rev=890056&r1=890055&r2=890056&view=diff
==============================================================================
--- cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java (original)
+++ cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java Sun Dec 13 12:16:17
2009
@@ -22,6 +22,7 @@
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.net.URL;
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.HashMap;
@@ -441,7 +442,7 @@
             Map<String, Object> reqContext = null;
             Map<String, Object> resContext = null;
             if (LOG.isLoggable(Level.FINE)) {
-                LOG.fine("Invoke, operation info: " + oi + ", params: " + params);
+                LOG.fine("Invoke, operation info: " + oi + ", params: " + Arrays.toString(params));
             }
             Message message = endpoint.getBinding().createMessage();
             if (null != context) {

Modified: cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java?rev=890056&r1=890055&r2=890056&view=diff
==============================================================================
--- cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
(original)
+++ cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
Sun Dec 13 12:16:17 2009
@@ -407,8 +407,8 @@
             if (LOG.isLoggable(Level.FINE)) {
                 LOG.log(Level.FINE, "Conduit '" + getConduitName()
                     + "' has been configured for TLS "
-                    + "keyManagers " + tlsClientParameters.getKeyManagers()
-                    + "trustManagers " + tlsClientParameters.getTrustManagers()
+                    + "keyManagers " + Arrays.toString(tlsClientParameters.getKeyManagers())
+                    + "trustManagers " + Arrays.toString(tlsClientParameters.getTrustManagers())
                     + "secureRandom " + tlsClientParameters.getSecureRandom()
                     + "Disable Common Name (CN) Check: " + tlsClientParameters.isDisableCNCheck());
             }
@@ -1379,8 +1379,8 @@
             if (LOG.isLoggable(Level.FINE)) {
                 LOG.log(Level.FINE, "Conduit '" + getConduitName()
                     + "' has been (re) configured for TLS "
-                    + "keyManagers " + tlsClientParameters.getKeyManagers()
-                    + "trustManagers " + tlsClientParameters.getTrustManagers()
+                    + "keyManagers " + Arrays.toString(tlsClientParameters.getKeyManagers())
+                    + "trustManagers " + Arrays.toString(tlsClientParameters.getTrustManagers())
                     + "secureRandom " + tlsClientParameters.getSecureRandom());
             }
             CertificateConstraintsType constraints = params.getCertConstraints();

Modified: cxf/trunk/tools/corba/src/main/java/org/apache/cxf/tools/corba/idlpreprocessor/DefaultIncludeResolver.java
URL: http://svn.apache.org/viewvc/cxf/trunk/tools/corba/src/main/java/org/apache/cxf/tools/corba/idlpreprocessor/DefaultIncludeResolver.java?rev=890056&r1=890055&r2=890056&view=diff
==============================================================================
--- cxf/trunk/tools/corba/src/main/java/org/apache/cxf/tools/corba/idlpreprocessor/DefaultIncludeResolver.java
(original)
+++ cxf/trunk/tools/corba/src/main/java/org/apache/cxf/tools/corba/idlpreprocessor/DefaultIncludeResolver.java
Sun Dec 13 12:16:17 2009
@@ -23,6 +23,7 @@
 import java.net.MalformedURLException;
 import java.net.URI;
 import java.net.URL;
+import java.util.Arrays;
 
 /**
  * @author lk
@@ -64,7 +65,8 @@
                 return url;
             } catch (MalformedURLException e) {
                 final PreprocessingException preprocessingException = new PreprocessingException(
-                        "Unable to resolve user include '" + spec + "' in '" + userIdlDirs
+ "'", null, 0);
+                    "Unable to resolve user include '" + spec + "' in '"
+                        + Arrays.toString(userIdlDirs) + "'", null, 0);
                 preprocessingException.initCause(e);
                 throw preprocessingException;
             }



Mime
View raw message