Author: scheu
Date: Sat Aug 14 13:49:22 2010
New Revision: 985491

URL: http://svn.apache.org/viewvc?rev=985491&view=rev
Log:
Debug trace changes only.

Modified:
    
axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingInHandler.java

Modified: 
axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingInHandler.java
URL: 
http://svn.apache.org/viewvc/axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingInHandler.java?rev=985491&r1=985490&r2=985491&view=diff
==============================================================================
--- 
axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingInHandler.java
 (original)
+++ 
axis/axis2/java/core/trunk/modules/addressing/src/org/apache/axis2/handlers/addressing/AddressingInHandler.java
 Sat Aug 14 13:49:22 2010
@@ -264,6 +264,9 @@ public class AddressingInHandler extends
         // Now that all the valid wsa headers have been read, throw an 
exception if there was an invalid cardinality
         // This means that if for example there are multiple MessageIDs and a 
FaultTo, the FaultTo will be respected.
         if (!duplicateHeaderNames.isEmpty()) {
+            if (log.isDebugEnabled()) {
+                log.debug("Duplicate header names found:" + 
duplicateHeaderNames.get(0));
+            }
             // Simply choose the first problem header we came across as we can 
only fault for one of them.
             
AddressingFaultsHelper.triggerInvalidCardinalityFault(messageContext,
                                                                   (String) 
duplicateHeaderNames
@@ -512,12 +515,16 @@ public class AddressingInHandler extends
         String wsaAction = soapHeaderBlock.getText();
 
         if (LoggingControl.debugLoggingAllowed && log.isTraceEnabled()) {
-            log.trace("extractActionInformation: soapAction='" + soapAction + 
"' wsa:Action='" +
-                      wsaAction + "'");
+            log.trace("extractActionInformation: HTTP soapAction or action ='" 
+ soapAction + 
+                    "' wsa:Action='" + wsaAction + "'");
         }
 
         // Need to validate that the content of the wsa:Action header is not 
null or whitespace
         if ((wsaAction == null) || "".equals(wsaAction.trim())) {
+            if (log.isDebugEnabled()) {
+                log.debug("The wsa:Action header is present but its contents 
are empty.  This violates " +
+                        "rules in the WS-A specification.  The SOAP node that 
sent this message must be changed.");
+            }
             
AddressingFaultsHelper.triggerActionNotSupportedFault(messageContext, 
wsaAction);
         }
 
@@ -529,6 +536,11 @@ public class AddressingInHandler extends
         // check that soapAction==wsa:Action
         if (soapAction != null && !"".equals(soapAction) && 
messageContext.isServerSide()) {
             if (!soapAction.equals(wsaAction)) {
+                if (log.isDebugEnabled()) {
+                    log.debug("The wsa:Action header is (" + wsaAction + ") 
which conflicts with the HTTP soapAction or action " +
+                            "(" + soapAction + ").  This is a violation of the 
WS-A specification.  The SOAP node that sent this message " +
+                            " must be changed.");
+                }
                 
AddressingFaultsHelper.triggerActionMismatchFault(messageContext, soapAction, 
wsaAction);
             }
         } else {


Reply via email to