Author: veithen
Date: Wed Apr 1 07:24:19 2009
New Revision: 760799
URL: http://svn.apache.org/viewvc?rev=760799&view=rev
Log:
Temporarily reverted the change in r759708, since this breaks sample 254. Needs
further investigation.
Modified:
synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/vfs/VFSTransportSender.java
Modified:
synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/vfs/VFSTransportSender.java
URL:
http://svn.apache.org/viewvc/synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/vfs/VFSTransportSender.java?rev=760799&r1=760798&r2=760799&view=diff
==============================================================================
---
synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/vfs/VFSTransportSender.java
(original)
+++
synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/vfs/VFSTransportSender.java
Wed Apr 1 07:24:19 2009
@@ -79,10 +79,16 @@
public void sendMessage(MessageContext msgCtx, String targetAddress,
OutTransportInfo outTransportInfo) throws AxisFault {
- if (waitForSynchronousResponse(msgCtx)) {
- throw new AxisFault("The VFS transport doesn't support synchronous
responses. " +
- "Please use the appropriate (out only) message exchange
pattern.");
- }
+ // TODO: this check is too strict and breaks sample 254
+ // Need to check
+ // - why this isn't detected by the unit tests
+ // - why this works with JMS (the JMS transport sender uses the same
method to determine
+ // whether to wait for a response; if this change breaks sample
254, one would also
+ // expect problems with sample 254 when substituting VFS by JMS)
+// if (waitForSynchronousResponse(msgCtx)) {
+// throw new AxisFault("The VFS transport doesn't support
synchronous responses. " +
+// "Please use the appropriate (out only) message exchange
pattern.");
+// }
VFSOutTransportInfo vfsOutInfo = null;
