cxf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Beryozkin (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CXF-5309) Support Bean Validation API for JAX-RS
Date Thu, 14 Nov 2013 12:59:22 GMT

    [ 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)

Mime
View raw message