reta commented on pull request #878:
URL: https://github.com/apache/cxf/pull/878#issuecomment-981960210


   @jgallimore thank you
   
   > Do we want to try and figure out a way to enable @encoded endpoints where 
we're reading from the HTTP parameter map? 
   
   At least for me, the uncertainty here is that we either "trust" the request 
params as being properly decoded or not. If we do, that basically means 
@Encoded is noop in this flow - we don't know if the request params were 
encoded in first place or not - the servlet hides this from us and just 
provides the final value. Contrary, if we get rid of this "trust" logic 
(basically, drop this pull request), then by adding `@Encoded` annotation to 
form params, the double decoding is also gone (to be fair, I would prefer to 
use only `@Encoded` as means to control decoding behavior).


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to