[ 
https://issues.apache.org/jira/browse/CXF-5309?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13822388#comment-13822388
 ] 

Sergey Beryozkin commented on CXF-5309:
---------------------------------------

Hi Andrei,

I applied part of your patch related to the name provider, I'll deal with the 
tests a bit later, I just want to add a JAXRSInvoker test there as well in 
order to test that the actual (per-request) service bean having the JAX-RS 
parameters injected into it can also be validated.

Re the name providers, I stopped short of introducing setters, they seemed a 
bit redundant to me, it does not appear any more difficult to use a 
'constructor-arg' instead of 'property', I exaggerated a bit there in my 
earlier comments. 

I also added ValidationConfiguration so that users can configure the factory as 
much as they like.  

Also removed the custom parameter info code from ValidationExceptionMapper, but 
also relaxed a bit the LOG level to WARNING, I guess we should use SEVERE only 
when the ValidatorProvider can not do its work

Thanks, Sergey 

> Support Bean Validation API for JAX-RS
> --------------------------------------
>
>                 Key: CXF-5309
>                 URL: https://issues.apache.org/jira/browse/CXF-5309
>             Project: CXF
>          Issue Type: Sub-task
>          Components: JAX-RS, JAX-WS Runtime
>            Reporter: Sergey Beryozkin
>         Attachments: patch-validation-exception-mapper.txt, 
> patch-validation-paramnameprovider.txt, patch-validation-poc.txt, 
> patch-validation-spring-tests.txt, patch-validation-tests.txt
>
>




--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to