JBAS-3734

[weston.price]

I think I understand why there is an issue, but I just wanted to verify that we want to do something like the following in the JmsServerSessions.onMessage() handling:

1) Check the current thread's context classloader
2) If it's null, set it to the CL of the ResourceAdapter prior to the beforeDelivery invocation
3) Set it back to the original CL after?

It's that last part that need some clarification on.

[adrian.brock]

No, the issue is the endpoint using whether the oldClassLoader
is null as the state variable meaning "beforeDeliveryInvoked".

In the case where the caller has no context classloader
(this came up where c threads are attached to the VM via JNI)
this test fails, because it thinks beforeDelivery wasn't invoked.

It just needs a seperate boolean "beforeDeliveryInvoked"
which it can check in afterDelivery().

[weston.price]

Ok, easy enough. So, to 'reclarify'

1) Simply add a state variable to the MessageEndpointInterceptor to say whether or not beforeDelivery was invoked.

2)Check variable in afterDelivery.

[adrian.brock]

Correct.

[weston.price]

Done.

[weston.price]

Should have also said that the variable needs to also be checked in the beforeDelivery method and replace the check of the oldCL.