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/

[bharath]

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;
             }