cxf-issues mailing list archives

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


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

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:
>             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

View raw message