Author: alessio.soldano(a)jboss.com Date: 2010-11-22 07:16:19 -0500 (Mon, 22 Nov 2010) New Revision: 13314 Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/client/ssl/SSLContextFactory.java stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/binding/MessageBinding.java stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/binding/PayloadBinding.java stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/EndpointImpl.java stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/ServiceDelegateImpl.java Log: [JBWS-2896] Fixing some log statements Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/client/ssl/SSLContextFactory.java =================================================================== --- stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/client/ssl/SSLContextFactory.java 2010-11-22 07:43:10 UTC (rev 13313) +++ stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/client/ssl/SSLContextFactory.java 2010-11-22 12:16:19 UTC (rev 13314) @@ -919,7 +919,7 @@ if (isServerSocketUseClientMode()) { keyManagers = null; - log.debug("Could not find keytore url. " + e.getMessage()); + log.debugf("Could not find keytore url. %s", e.getMessage()); } else { @@ -938,7 +938,7 @@ catch (NullStoreURLException e) { trustManagers = null; - log.debug("Could not find truststore url. " + e.getMessage()); + log.debugf("Could not find truststore url. %s", e.getMessage()); } secureRandom = getSecureRandom(); @@ -982,7 +982,7 @@ { // this is allowable since would be the normal scenario keyManagers = null; - log.debug("Could not find keystore url. " + e.getMessage()); + log.debugf("Could not find keystore url. %s ", e.getMessage()); } try @@ -998,7 +998,7 @@ if(keyManagers != null) { trustManagers = null; - log.debug("Could not find truststore url. " + e.getMessage()); + log.debugf("Could not find truststore url. %s", e.getMessage()); } else { Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/binding/MessageBinding.java =================================================================== --- stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/binding/MessageBinding.java 2010-11-22 07:43:10 UTC (rev 13313) +++ stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/binding/MessageBinding.java 2010-11-22 12:16:19 UTC (rev 13314) @@ -67,7 +67,7 @@ /** On the server side, extract the IN parameters from the payload and populate an Invocation object */ public EndpointInvocation unbindRequestMessage(OperationMetaData opMetaData, MessageAbstraction reqMessage) throws BindingException { - log.debug("unbindRequestMessage: " + opMetaData.getQName()); + log.debugf("unbindRequestMessage: %s", opMetaData.getQName()); try { // Construct the endpoint invocation object @@ -93,7 +93,7 @@ /** On the server side, generate the payload from OUT parameters. */ public MessageAbstraction bindResponseMessage(OperationMetaData opMetaData, EndpointInvocation epInv) throws BindingException { - log.debug("bindResponseMessage: " + opMetaData.getQName()); + log.debugf("bindResponseMessage: %s", opMetaData.getQName()); try { SOAPMessageContextJAXWS msgContext = (SOAPMessageContextJAXWS)MessageContextAssociation.peekMessageContext(); Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/binding/PayloadBinding.java =================================================================== --- stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/binding/PayloadBinding.java 2010-11-22 07:43:10 UTC (rev 13313) +++ stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/binding/PayloadBinding.java 2010-11-22 12:16:19 UTC (rev 13314) @@ -78,7 +78,7 @@ /** On the server side, extract the IN parameters from the payload and populate an Invocation object */ public EndpointInvocation unbindRequestMessage(OperationMetaData opMetaData, MessageAbstraction payload) throws BindingException { - log.debug("unbindRequestMessage: " + opMetaData.getQName()); + log.debugf("unbindRequestMessage: %s", opMetaData.getQName()); try { // Construct the endpoint invocation object @@ -119,7 +119,7 @@ /** On the server side, generate the payload from OUT parameters. */ public MessageAbstraction bindResponseMessage(OperationMetaData opMetaData, EndpointInvocation epInv) throws BindingException { - log.debug("bindResponseMessage: " + opMetaData.getQName()); + log.debugf("bindResponseMessage: %s", opMetaData.getQName()); try { Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/EndpointImpl.java =================================================================== --- stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/EndpointImpl.java 2010-11-22 07:43:10 UTC (rev 13313) +++ stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/EndpointImpl.java 2010-11-22 12:16:19 UTC (rev 13314) @@ -145,7 +145,8 @@ if (context == null) throw new IllegalArgumentException("Null context"); - log.debug("publishing endpoint " + this + " to " + context); + if (log.isDebugEnabled()) + log.debug("publishing endpoint " + this + " to " + context); if (isDestroyed) throw new IllegalStateException("Endpoint already destroyed"); Modified: stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/ServiceDelegateImpl.java =================================================================== --- stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/ServiceDelegateImpl.java 2010-11-22 07:43:10 UTC (rev 13313) +++ stack/native/trunk/modules/core/src/main/java/org/jboss/ws/core/jaxws/spi/ServiceDelegateImpl.java 2010-11-22 12:16:19 UTC (rev 13314) @@ -435,7 +435,7 @@ if (usRef == null) { - log.debug("No port configuration for: " + portName); + log.debugf("No port configuration for: %s", portName); return; } @@ -454,14 +454,14 @@ Map<String, Object> reqCtx = bp.getRequestContext(); for (UnifiedStubPropertyMetaData prop : pcref.getStubProperties()) { - log.debug("Set stub property: " + prop); + log.debugf("Set stub property: %s", prop); reqCtx.put(prop.getPropName(), prop.getPropValue()); } } if (configName != null || configFile != null) { - log.debug("Configure Stub: [configName=" + configName + ",configFile=" + configFile + "]"); + log.debugf("Configure Stub: [configName=%s,configFile=%s]", configName, configFile); stub.setConfigName(configName, configFile); } }