Issue Details (XML | Word | Printable)

Key: MULE-927
Type: Patch submission Patch submission
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Ross Mason
Reporter: Jesper Steen Møller
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Mule

Axis connector uses local logic to set sync/asynchronous messaging instead of using environment's default setting

Created: 24/Jul/06 07:59 AM   Updated: 30/Nov/08 03:52 AM
Component/s: Transport: Axis
Affects Version/s: 1.3-rc4
Fix Version/s: 2.0.0

Time Tracking:
Not Specified

File Attachments: 1. Text File MULE-927-patch.txt (0.6 kB)

Environment: Axis used with the servlet provider

Labels:


 Description  « Hide
When registering a receiver with the mule service, the Axis connector assumes that the endpoint is always async unless if the 'synchronous' property is explicitly set or if the protocol is one of a select few (http, https, tcp or ssl). Thus, when using the servlet provider, in an otherwise synchronous environment, axis services suddenly act in an asynchronous fashion.

The offending code is in AxisConnector.registerReceiverWithMuleService (excerpt):

//Default to using synchronous for socket based protocols unless the
//synchronous property has been set explicitly
boolean sync = false;
if(!receiver.getEndpoint().isSynchronousSet()) {
if (scheme.equals("http") || scheme.equals("https") || scheme.equals("ssl") || scheme.equals("tcp")) { sync = true; }
} else { sync = receiver.getEndpoint().isSynchronous(); }

I think the best solution is to use the environment properties' default value for synchronous instead - that's the common concensus. Perhaps that way, the explicit checks for "protocols known to be synchronous" could be done away with, but I think that's secondary)

I haven't checked XFire or Glue implementations to see if they have similar issues.

If you agree to the fix, I can make the change.



 All   Comments   Work Log   Change History   Transitions   FishEye      Sort Order: Ascending order - Click to sort in descending order
Jesper Steen Møller added a comment - 25/Jul/06 03:41 AM
A simple patch to apply to /trunk/mule/providers/axis/src/main/java/org/mule/providers/soap/axis
This is the more gentle (safest) of the proposed strategies.

I can't think of any possible negative side effects. Please review!