You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by GitBox <gi...@apache.org> on 2021/04/14 02:33:16 UTC
[GitHub] [cxf] andymc12 opened a new pull request #778: [CXF-7996] Fix `@Encode` TCK test
andymc12 opened a new pull request #778:
URL: https://github.com/apache/cxf/pull/778
This should fix the following Jakarta RESTful WS TCK failures:
> com/sun/ts/tests/jaxrs/ee/rs/beanparam/form/plain/JAXRSClient.java#formFieldParamEntityWithEncodedTest_from_standalone: JAXRSClient_formFieldParamEntityWithEncodedTest_from_standalone
> com/sun/ts/tests/jaxrs/ee/rs/beanparam/form/plain/JAXRSClient.java#formParamEntityWithEncodedTest_from_standalone: JAXRSClient_formParamEntityWithEncodedTest_from_standalone
The issue is that (prior to this fix) form parameters are read into a map as either decoded or encoded based on whether the first parameter has the `@Encoded` annotation or not. This can result in the `@Encoded` annotation on second or subsequent parameters getting ignored.
The fix is to read the parameters into the map without performing any decoding until it is determined which parameters should be decoded.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cxf] dblevins commented on pull request #778: [CXF-7996] Fix `@Encode` TCK test
Posted by GitBox <gi...@apache.org>.
dblevins commented on pull request #778:
URL: https://github.com/apache/cxf/pull/778#issuecomment-821471614
Thank you @andymc12 !
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cxf] andymc12 merged pull request #778: [CXF-7996] Fix `@Encode` TCK test
Posted by GitBox <gi...@apache.org>.
andymc12 merged pull request #778:
URL: https://github.com/apache/cxf/pull/778
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [cxf] andymc12 commented on pull request #778: [CXF-7996] Fix `@Encode` TCK test
Posted by GitBox <gi...@apache.org>.
andymc12 commented on pull request #778:
URL: https://github.com/apache/cxf/pull/778#issuecomment-821434796
Should resolve https://issues.apache.org/jira/browse/CXF-8461 and https://issues.apache.org/jira/browse/CXF-8462
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org