[jbossws-commits] [jbossws/jbossws-cxf] fbbf2c: [JBWS-4444] Server throws IllegalStateException wh...

Branch: refs/heads/jbossws-cxf-5.4.x Home: https://github.com/jbossws/jbossws-cxf Commit: fbbf2c93644601b177dea6315b3eee0ecb14d4d6 https://github.com/jbossws/jbossws-cxf/commit/fbbf2c93644601b177dea6315b3... Author: Richard Opálka <opalka.richard(a)gmail.com&gt; Date: 2025-05-05 (Mon, 05 May 2025) Changed paths: M docbook/src/main/doc/adoc/content/chapter-5-Advanced_User_Guide.adoc M docbook/src/main/doc/chapter-5-Advanced_User_Guide.xml M modules/server/src/main/java/org/jboss/wsf/stack/cxf/configuration/ServerBeanCustomizer.java A modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/AbstractTCCLAwarePhaseInterceptor.java M modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/AbstractTCCLPhaseInterceptor.java M modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/HandlerConfigInterceptor.java A modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/TCCLAwareInterceptorReplacer.java A modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/TCCLAwarePhaseInterceptor.java A modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/TCCLAwareSoapPhaseInterceptor.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/CDIOutInterceptor.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/HelloBean.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/JBWS4430TestCase.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/LoggingHandler.java Log Message: ----------- [JBWS-4444] Server throws IllegalStateException when call to SOAPHandler's getHeaders/handleFault/close methods is performed and their code include the CDI bean invocation The idea of the fix is the following: * Every non Apache CXF interceptor is considered either JBossWS provided or user provided interceptor * Each such non Apache CXF interceptor is wrapped with TCCL aware interceptor - this wrapping is necessary for all Endpoint, Service, Bus & Binding interceptors - every TCCL aware interceptor delegates its calls (with proper TCCL in place where necessary) to wrapped delegate interceptor * The only exception from Apaches's CXF interceptors is: org.apache.cxf.jaxws.handler.soap.SOAPHandlerInterceptor - that one needs special treatment and must be wrapped too (luckily there was no cast in Apache CXF code base involving this one - otherwise it wouldn't work) * Special treatment is needed also for org.apache.cxf.binding.soap.interceptor.SoapInterceptor instances - to delegate properly TCCL aware calls to getUnderstoodHeaders() method * org.jboss.wsf.stack.cxf.interceptor.AbstractTCCLPhaseInterceptor was marked as deprecated, its usages inside code base have been removed and documentation referencing it was eliminated Commit: 075e4d8a0cdaaee5e92f507a5ef09ed4ddceb381 https://github.com/jbossws/jbossws-cxf/commit/075e4d8a0cdaaee5e92f507a5ef... Author: bmaxwell <bmaxwell(a)redhat.com&gt; Date: 2025-05-05 (Mon, 05 May 2025) Changed paths: A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/AccessTokenClientHandler.java A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/AccessTokenClientHandlerResolver.java A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/Client.java A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/ClientBean.java A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/CredentialsCDIBean.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/JBWS4430TestCase.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/LoggingHandler.java A modules/testsuite/cxf-tests/src/test/resources/jaxws/cxf/jbws4430/WEB-INF/beans.xml A modules/testsuite/cxf-tests/src/test/resources/jaxws/cxf/jbws4430/WEB-INF/wsdl/ClientBeanService.wsdl M modules/testsuite/pom.xml M pom.xml Log Message: ----------- [JBWS-4444] add custom HandlerResolver to JBWS4430TestCase client test, added beans.xml, added ClientBean intermediate to test CDI in client custom HandlerResolver Commit: dc4713676c0aa6aa70674e3af104a38ffdf829f0 https://github.com/jbossws/jbossws-cxf/commit/dc4713676c0aa6aa70674e3af10... Author: Richard Opálka <opalka.richard(a)gmail.com&gt; Date: 2025-05-05 (Mon, 05 May 2025) Changed paths: M modules/client/src/main/java/org/jboss/wsf/stack/cxf/client/ProviderImpl.java A modules/client/src/main/java/org/jboss/wsf/stack/cxf/client/TCCLAwareHandler.java A modules/client/src/main/java/org/jboss/wsf/stack/cxf/client/TCCLAwareHandlerReplacer.java A modules/client/src/main/java/org/jboss/wsf/stack/cxf/client/TCCLAwareHandlerResolver.java M modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/AbstractTCCLAwarePhaseInterceptor.java M modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/HandlerConfigInterceptor.java M modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/TCCLAwareSoapPhaseInterceptor.java Log Message: ----------- [JBWS-4444] Server throws IllegalStateException when call to SOAPHandler's getHeaders/handleFault/close ethods is peformed and their code include the CDI bean invocation - second round of fixes The idea of this second round of fixes is the following: * setHandlerResolver() of provider implementation is overriden to wrap client provided handler resolver with TCCL aware handler resolver * TCCL aware Handler resolver wraps every handler client provided with TCCL aware handler * finally there's a minor refactoring included that uses instanceof check against DelegateClassLoader instead of implementation specific JAXPDelegateClassLoader Commit: 2f8bd8b0ef514fe797c4fb05119aff71d5a6ecb3 https://github.com/jbossws/jbossws-cxf/commit/2f8bd8b0ef514fe797c4fb05119... Author: Jim Ma <ema(a)redhat.com&gt; Date: 2025-05-05 (Mon, 05 May 2025) Changed paths: M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/JBWS4430TestCase.java Log Message: ----------- [Minor Change]:Remove ZipExporter in test Commit: aff400ebd3ce11ba98b5b28612545aff6ca754f2 https://github.com/jbossws/jbossws-cxf/commit/aff400ebd3ce11ba98b5b286125... Author: Richard Opálka <opalka.richard(a)gmail.com&gt; Date: 2025-05-05 (Mon, 05 May 2025) Changed paths: M docbook/src/main/doc/adoc/content/chapter-5-Advanced_User_Guide.adoc M docbook/src/main/doc/chapter-5-Advanced_User_Guide.xml M modules/client/src/main/java/org/jboss/wsf/stack/cxf/client/ProviderImpl.java A modules/client/src/main/java/org/jboss/wsf/stack/cxf/client/TCCLAwareHandler.java A modules/client/src/main/java/org/jboss/wsf/stack/cxf/client/TCCLAwareHandlerReplacer.java A modules/client/src/main/java/org/jboss/wsf/stack/cxf/client/TCCLAwareHandlerResolver.java M modules/server/src/main/java/org/jboss/wsf/stack/cxf/configuration/ServerBeanCustomizer.java A modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/AbstractTCCLAwarePhaseInterceptor.java M modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/AbstractTCCLPhaseInterceptor.java M modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/HandlerConfigInterceptor.java A modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/TCCLAwareInterceptorReplacer.java A modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/TCCLAwarePhaseInterceptor.java A modules/server/src/main/java/org/jboss/wsf/stack/cxf/interceptor/TCCLAwareSoapPhaseInterceptor.java A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/AccessTokenClientHandler.java A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/AccessTokenClientHandlerResolver.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/CDIOutInterceptor.java A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/Client.java A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/ClientBean.java A modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/CredentialsCDIBean.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/HelloBean.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/JBWS4430TestCase.java M modules/testsuite/cxf-tests/src/test/java/org/jboss/test/ws/jaxws/cxf/jbws4430/LoggingHandler.java A modules/testsuite/cxf-tests/src/test/resources/jaxws/cxf/jbws4430/WEB-INF/beans.xml A modules/testsuite/cxf-tests/src/test/resources/jaxws/cxf/jbws4430/WEB-INF/wsdl/ClientBeanService.wsdl M modules/testsuite/pom.xml M pom.xml Log Message: ----------- Merge pull request #593 from spyrkob/JBEAP-28792_illegal_state_exception [JBEAP-28792] Server throws IllegalStateException when call to SOAPHandler's getHeaders/handleFault/close ethods is peformed and their code include the CDI bean invocation Compare: https://github.com/jbossws/jbossws-cxf/compare/4bcf61fa3e9b...aff400ebd3ce To unsubscribe from these emails, change your notification settings at https://github.com/jbossws/jbossws-cxf/settings/notifications