You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by bh...@apache.org on 2009/12/13 13:16:17 UTC
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/
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;
}