[GitHub] [cxf] ilgrosso commented on a change in pull request #926: [CXF-8680] Encapsulating getter method resolution in ClientProxyImpl

2022-03-22 Thread GitBox
ilgrosso commented on a change in pull request #926: URL: https://github.com/apache/cxf/pull/926#discussion_r831828614 ## File path: rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProxyImpl.java ## @@ -407,6 +410,23 @@ protected static int getBodyIndex(Multivalu

[GitHub] [cxf] ilgrosso commented on a change in pull request #926: [CXF-8680] Encapsulating getter method resolution in ClientProxyImpl

2022-03-22 Thread GitBox
ilgrosso commented on a change in pull request #926: URL: https://github.com/apache/cxf/pull/926#discussion_r831828614 ## File path: rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProxyImpl.java ## @@ -407,6 +410,23 @@ protected static int getBodyIndex(Multivalu

Re: Systest failures with wss4j-3.0.0-SNAPSHOT

2022-03-22 Thread Colm O hEigeartaigh
Hi Jim, Which branch are you testing this on in CXF? Colm. On Mon, Mar 21, 2022 at 7:26 AM Jim Ma wrote: > > Hi Colm, > After I upgraded wss4j version to 3.0.0-SNAPSHOT(jakarta version) in CXF, > I got the following ws-security systests failures. > Could you please help look at if there is sti

[GitHub] [cxf] reta commented on a change in pull request #926: [CXF-8680] Encapsulating getter method resolution in ClientProxyImpl

2022-03-22 Thread GitBox
reta commented on a change in pull request #926: URL: https://github.com/apache/cxf/pull/926#discussion_r832066799 ## File path: rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProxyImpl.java ## @@ -407,6 +410,23 @@ protected static int getBodyIndex(MultivaluedMa

[GitHub] [cxf] ilgrosso merged pull request #926: [CXF-8680] Encapsulating getter method resolution in ClientProxyImpl

2022-03-22 Thread GitBox
ilgrosso merged pull request #926: URL: https://github.com/apache/cxf/pull/926 -- 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 specific comment. To unsubscribe, e-mail: dev-unsubscr...@cxf

[GitHub] [cxf] ilgrosso commented on pull request #926: [CXF-8680] Encapsulating getter method resolution in ClientProxyImpl

2022-03-22 Thread GitBox
ilgrosso commented on pull request #926: URL: https://github.com/apache/cxf/pull/926#issuecomment-1075067910 Thanks for review @reta ! -- 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 specifi

Re: Systest failures with wss4j-3.0.0-SNAPSHOT

2022-03-22 Thread Jim Ma
Hi Colm, Please checkout my working branch : https://github.com/jimma/cxf/tree/wss4j-30-jakarta Build all cxf artifacts first with: mvn clean install -Pfastinstall then go to systests/ws-security, run these tests and you'll get these test failures. Please let me know if you have other issues when