|
[
Permlink
| « Hide
]
Daniel Feist added a comment - 15/May/08 06:51 AM
Can you comment on this please Ross?
I think there is a wider issue here (though I don't fully understand form your description). I goes back to the Messaging styles and the behaviour we expect from each. I took a look at the diagrams that you did, but I don't think they are accurate. We need to define each message style, it's variations and come up with consistent rules abuout what happens in each scenario.
Agreed. Even so I'm convinced there is a bug here such that one of the messaging styles is not behaving as it should be and as it used to in 1.4 due to some messaging api changes. We'll need to fix this in the short term. I'll try to update this issue with a better description of the bug, and reference the failing test case.
This patch fixes the issue, but it's more message copying..
Index: /Users/dfeist/MULE/2.0.X_BRANCH/core/src/main/java/org/mule/service/AbstractService.java
===================================================================
--- /Users/dfeist/MULE/2.0.X_BRANCH/core/src/main/java/org/mule/service/AbstractService.java (revision 11844)
+++ /Users/dfeist/MULE/2.0.X_BRANCH/core/src/main/java/org/mule/service/AbstractService.java (working copy)
@@ -10,6 +10,7 @@
package org.mule.service;
+import org.mule.DefaultMuleMessage;
import org.mule.OptimizedRequestContext;
import org.mule.api.MessagingException;
import org.mule.api.MuleContext;
@@ -875,7 +876,7 @@
{
if (getOutboundRouter().hasEndpoints())
{
- getOutboundRouter().route(result, event.getSession(), event.isSynchronous());
+ getOutboundRouter().route(new DefaultMuleMessage(result), event.getSession(), event.isSynchronous());
}
}
}
@@ -890,7 +891,7 @@
{
if (getOutboundRouter().hasEndpoints())
{
- MuleMessage outboundReturnMessage = getOutboundRouter().route(result, event.getSession(),
+ MuleMessage outboundReturnMessage = getOutboundRouter().route(new DefaultMuleMessage(result), event.getSession(),
event.isSynchronous());
if (outboundReturnMessage != null)
{
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
 |
|
Bug
Closed
Blocker
Response messages in remoteSync=false scenarios are corrupted by transformations that happen in outbound phase
)
