Author: ruwan
Date: Wed Sep 23 03:05:20 2009
New Revision: 817943

URL: http://svn.apache.org/viewvc?rev=817943&view=rev
Log:
Partial refactoring to get the WSCOMMONS-444 fixed (it requires synapse changes 
as well)

Modified:
    
webservices/commons/branches/modules/transport/1.0.0/modules/base/src/main/java/org/apache/axis2/transport/base/AbstractTransportSender.java
    
webservices/commons/branches/modules/transport/1.0.0/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java

Modified: 
webservices/commons/branches/modules/transport/1.0.0/modules/base/src/main/java/org/apache/axis2/transport/base/AbstractTransportSender.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/branches/modules/transport/1.0.0/modules/base/src/main/java/org/apache/axis2/transport/base/AbstractTransportSender.java?rev=817943&r1=817942&r2=817943&view=diff
==============================================================================
--- 
webservices/commons/branches/modules/transport/1.0.0/modules/base/src/main/java/org/apache/axis2/transport/base/AbstractTransportSender.java
 (original)
+++ 
webservices/commons/branches/modules/transport/1.0.0/modules/base/src/main/java/org/apache/axis2/transport/base/AbstractTransportSender.java
 Wed Sep 23 03:05:20 2009
@@ -134,7 +134,6 @@
         MessageContext msgCtx, Map trpHeaders,
         String soapAction, String contentType) {
 
-
         // set the soapaction if one is available via a transport header
         if (soapAction != null) {
             msgCtx.setSoapAction(soapAction);

Modified: 
webservices/commons/branches/modules/transport/1.0.0/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/branches/modules/transport/1.0.0/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java?rev=817943&r1=817942&r2=817943&view=diff
==============================================================================
--- 
webservices/commons/branches/modules/transport/1.0.0/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java
 (original)
+++ 
webservices/commons/branches/modules/transport/1.0.0/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java
 Wed Sep 23 03:05:20 2009
@@ -449,22 +449,14 @@
         // load any transport headers from received message
         JMSUtils.loadTransportHeaders(message, responseMsgCtx);
 
-        // workaround for Axis2 TransportUtils.createSOAPMessage() issue, 
where a response
-        // of content type "text/xml" is thought to be REST if 
!MC.isServerSide(). This
-        // question is still under debate and due to the timelines, I am 
commiting this
-        // workaround as Axis2 1.2 is about to be released and Synapse 1.0
-        responseMsgCtx.setServerSide(false);
-
-        String contentType =
-                contentTypeProperty == null ? null
-                        : JMSUtils.getProperty(message, contentTypeProperty);
+        String contentType = contentTypeProperty == null ? null
+                : JMSUtils.getProperty(message, contentTypeProperty);
 
         try {
             JMSUtils.setSOAPEnvelope(message, responseMsgCtx, contentType);
         } catch (JMSException ex) {
             throw AxisFault.makeFault(ex);
         }
-//        responseMsgCtx.setServerSide(true);
 
         handleIncomingMessage(
             responseMsgCtx,


Reply via email to