Re: CXF-8378: Backward compatible issue in 3.4.1 and 3.3.8

2020-11-22 Thread Jim Ma
Hi Colm, I think this commit [1] caused the regression. We have some tests for CXFAuthenticator in CXF, but they are running with the single "flat" classloader and don't throw this exception. [1] https://github.com/apache/cxf/commit/58539be7c6367b0e7db354cd90467fe006ddef57 Cheers, Jim On Fri,

[GitHub] [cxf] rmannibucau commented on a change in pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
rmannibucau commented on a change in pull request #721: URL: https://github.com/apache/cxf/pull/721#discussion_r528502320 ## File path: rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperClassLoader.java ## @@ -0,0 +1,57 @@ +/** + * Licensed to the Apache

[GitHub] [cxf] reta commented on a change in pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
reta commented on a change in pull request #721: URL: https://github.com/apache/cxf/pull/721#discussion_r528443595 ## File path: rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperClassLoader.java ## @@ -0,0 +1,57 @@ +/** + * Licensed to the Apache Software

[GitHub] [cxf] reta commented on a change in pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
reta commented on a change in pull request #721: URL: https://github.com/apache/cxf/pull/721#discussion_r528443595 ## File path: rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperClassLoader.java ## @@ -0,0 +1,57 @@ +/** + * Licensed to the Apache Software

[GitHub] [cxf] reta commented on a change in pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
reta commented on a change in pull request #721: URL: https://github.com/apache/cxf/pull/721#discussion_r528443595 ## File path: rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperClassLoader.java ## @@ -0,0 +1,57 @@ +/** + * Licensed to the Apache Software

[GitHub] [cxf] dufoli commented on a change in pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
dufoli commented on a change in pull request #721: URL: https://github.com/apache/cxf/pull/721#discussion_r528427949 ## File path: rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperClassLoader.java ## @@ -0,0 +1,57 @@ +/** + * Licensed to the Apache Software

[GitHub] [cxf] reta edited a comment on pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
reta edited a comment on pull request #721: URL: https://github.com/apache/cxf/pull/721#issuecomment-731858336 @dufoli so I figured it out, a number of issues here: 1. `java.lang.VerifyError: (class: org/apache/cxf/jaxws/service/jaxws_asm/AddNumbers_WrapperTypeHelper1, method:

[GitHub] [cxf] reta commented on a change in pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
reta commented on a change in pull request #721: URL: https://github.com/apache/cxf/pull/721#discussion_r528415313 ## File path: rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/WrapperHelperClassLoader.java ## @@ -0,0 +1,57 @@ +/** + * Licensed to the Apache Software

[GitHub] [cxf] reta commented on pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
reta commented on pull request #721: URL: https://github.com/apache/cxf/pull/721#issuecomment-731858336 @dufoli so I figured it out, a number of issues here: 1. `java.lang.VerifyError: (class: org/apache/cxf/jaxws/service/jaxws_asm/AddNumbers_WrapperTypeHelper1, method:

[GitHub] [cxf] dufoli commented on pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
dufoli commented on pull request #721: URL: https://github.com/apache/cxf/pull/721#issuecomment-731844744 @reta I have check and it is quite the same code (just few adaptation) so it must come from opcodes mapping but it is strange because other test pass. So maybe issue is more with

[GitHub] [cxf] reta commented on pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
reta commented on pull request #721: URL: https://github.com/apache/cxf/pull/721#issuecomment-731807241 Interesting, the bytecode generation part supposed to be the same, do you need help troubleshooting this one, @dufoli ?

[GitHub] [cxf] dufoli commented on pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
dufoli commented on pull request #721: URL: https://github.com/apache/cxf/pull/721#issuecomment-731798422 in WrapperHelperCompiler debug I have an exception. java.lang.VerifyError: (class: org/apache/cxf/jaxws/service/jaxws_asm/AddNumbers_WrapperTypeHelper1, method:

[GitHub] [cxf] dufoli commented on pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
dufoli commented on pull request #721: URL: https://github.com/apache/cxf/pull/721#issuecomment-731782694 I have an error related to classLoader during build:: ``` [WARNING] Error during type resolution of field 'binding' in class org.apache.cxf.jaxws.DispatchImpl due to:

[GitHub] [cxf] reta commented on pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
reta commented on pull request #721: URL: https://github.com/apache/cxf/pull/721#issuecomment-731767497 @dufoli sorry for the hiccups, I asked for help from Apache Infra team to understand why Jenkins checks are not being triggered (sadly have limited access to Github repository setting),

[GitHub] [cxf] reta merged pull request #731: Update to Swagger UI 3.37.0

2020-11-22 Thread GitBox
reta merged pull request #731: URL: https://github.com/apache/cxf/pull/731 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [cxf] reta commented on pull request #730: Cxf 8340

2020-11-22 Thread GitBox
reta commented on pull request #730: URL: https://github.com/apache/cxf/pull/730#issuecomment-731759040 @dufoli sorry, will close this one, trying to revive https://github.com/apache/cxf/pull/721 This is an automated

[GitHub] [cxf] reta closed pull request #730: Cxf 8340

2020-11-22 Thread GitBox
reta closed pull request #730: URL: https://github.com/apache/cxf/pull/730 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [cxf] reta opened a new pull request #731: Update to Swagger UI 3.37.0

2020-11-22 Thread GitBox
reta opened a new pull request #731: URL: https://github.com/apache/cxf/pull/731 Update to Swagger UI 3.37.0 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [cxf] dufoli opened a new pull request #730: Cxf 8340

2020-11-22 Thread GitBox
dufoli opened a new pull request #730: URL: https://github.com/apache/cxf/pull/730 close #721 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [cxf] dufoli closed pull request #721: [cxf-8340] add precompiled class for Graalvm native support

2020-11-22 Thread GitBox
dufoli closed pull request #721: URL: https://github.com/apache/cxf/pull/721 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the