Hi all, before publishing next release, any comment on that point ?
Regards, Jean-Louis Jean-Louis MONTEIRO wrote: > > Hi, > > I had a look on the post > http://www.nabble.com/exceptions-handling-with-Webservices-td19668275.html > http://www.nabble.com/exceptions-handling-with-Webservices-td19668275.html > . > > I have a question related to the EjbInterceptor class (openejb-cxf > module). > In the invoke method, we can either directly call the target EJB bean > (when no handler has been defined) or delegate to the CXF chain when > handlers have been defined. > > Regarding the first case, when an ApplicationException is thrown. This > exception is caught by the preEjbInvoke method (from the > EjbMethodInvoker). The preEjbInvoke method registers all exceptions as > UNCHECKED so the client always receives a SOAPFaultException instead of > the user defined application exception. > > We can work around this problem by adding a dummy handler or change the > test to be sure it's always false. > > To conclude, the optimization made to directly invoke the target EJB bean > does not handle exceptions correctly. > > Any input is welcome. > Kind regards, > Jean-Louis > > > > David Blevins wrote: >> >> Good catch! Went ahead and made that change. >> >> -David >> >> On Sep 25, 2008, at 11:54 AM, Jean-Louis MONTEIRO wrote: >> >>> >>> Hi all, >>> >>> I'm facing a problem with application exceptions handling in OpenEJB >>> (trunk >>> version). >>> Just for information, it worked on previous version (3.0). >>> >>> It's directly link with the point "14.2.1 Application Exception" >>> from the >>> last EJB 3.0 specification. >>> The AnnotationDeployer (from the core module) does not include >>> application >>> exception (with ApplicationException annotation). >>> >>> I think the problem is more or less at line 429 because the test is >>> always >>> false (except when the application exception is declared in the ejb- >>> jar.xml >>> file). >>> Should the test be ? >>> if (assemblyDescriptor.getApplicationException(exceptionClass) == >>> null) { >>> >>> Kind regards, >>> Jean-Louis >>> >>> >>> >>> >>> >>> >>> -- >>> View this message in context: >>> http://www.nabble.com/Application-exception-handling-problem-tp19673820p19673820.html >>> Sent from the OpenEJB Dev mailing list archive at Nabble.com. >>> >>> >> >> >> > > -- View this message in context: http://www.nabble.com/Application-exception-handling-problem-tp19673820p19987699.html Sent from the OpenEJB Dev mailing list archive at Nabble.com.
