You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by sc...@apache.org on 2009/06/17 16:53:16 UTC
svn commit: r785637 -
/webservices/axis2/trunk/java/modules/transport/http/src/org/apache/axis2/transport/http/CommonsHTTPTransportSender.java
Author: scheu
Date: Wed Jun 17 14:53:16 2009
New Revision: 785637
URL: http://svn.apache.org/viewvc?rev=785637&view=rev
Log:
JIRA:AXIS2-4264
Contributor:Rich Scheuerle and Davanum Srinivas
Summary:
Don't expose internal (anon*) soap actions.
Modified:
webservices/axis2/trunk/java/modules/transport/http/src/org/apache/axis2/transport/http/CommonsHTTPTransportSender.java
Modified: webservices/axis2/trunk/java/modules/transport/http/src/org/apache/axis2/transport/http/CommonsHTTPTransportSender.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/transport/http/src/org/apache/axis2/transport/http/CommonsHTTPTransportSender.java?rev=785637&r1=785636&r2=785637&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/transport/http/src/org/apache/axis2/transport/http/CommonsHTTPTransportSender.java (original)
+++ webservices/axis2/trunk/java/modules/transport/http/src/org/apache/axis2/transport/http/CommonsHTTPTransportSender.java Wed Jun 17 14:53:16 2009
@@ -23,6 +23,7 @@
import org.apache.axis2.AxisFault;
import org.apache.axis2.Constants;
import org.apache.axis2.addressing.EndpointReference;
+import org.apache.axis2.client.ServiceClient;
import org.apache.axis2.context.ConfigurationContext;
import org.apache.axis2.context.MessageContext;
import org.apache.axis2.description.Parameter;
@@ -418,12 +419,27 @@
if ((soapActionString == null) || (soapActionString.length() == 0)) {
// now let's try to get WSA action
soapActionString = messageContext.getWSAAction();
+ if (log.isDebugEnabled()) {
+ log.debug("SOAP Action from getWSAAction was : " + soapActionString);
+ }
if (messageContext.getAxisOperation() != null
&& ((soapActionString == null) || (soapActionString
.length() == 0))) {
// last option is to get it from the axis operation
- soapActionString = messageContext.getAxisOperation()
- .getSoapAction();
+ String axisOpSOAPAction = messageContext.getAxisOperation().
+ getSoapAction();
+ if (log.isDebugEnabled()) {
+ log.debug("SOAP Action from AxisOperation was : " + axisOpSOAPAction);
+ }
+ if (ServiceClient.ANON_OUT_ONLY_OP.equals(axisOpSOAPAction)
+ || (ServiceClient.ANON_OUT_ONLY_OP.equals(axisOpSOAPAction))
+ || (ServiceClient.ANON_OUT_ONLY_OP.equals(axisOpSOAPAction))) {
+ if (log.isDebugEnabled()) {
+ log.debug("Will not override SOAP Action because " + axisOpSOAPAction + " in AxisOperation was auto-generated");
+ }
+ } else {
+ soapActionString = axisOpSOAPAction;
+ }
}
}
}