Hi Francesco,

Happy to be the good one :-)
Please see comments inline
On 22/09/16 12:07, Francesco Chicchiriccò wrote:
Hi all,
during a recent chat I had with the good Sergey (on another topic), I
have found that the JAX-RS's standard WebApplicationException (and child
exceptions) are handled by CXF, according to the specs, via
WebApplicationExceptionMapper [1].

Syncope's RestServiceExceptionMapper, however, contains some code [2] to
manage such exceptions, which is never going to be triggered, at this
point.

Please consider that this affects both 1.2 and 2.0.

IMO, we should either:

 (a) remove [2] and let CXF do its work according to the specs
 (b) set the 'default.wae.mapper.least.specific' (which is false by
default) to delegate WebApplicationException to [2]

I've checked the spec archives - indeed Marek was saying that WAE, being a special type of exception, should not be given to the mappers which are typed to handle WAE subclasses (in that case it was a RuntimeException mapper).
CXF needs to have a WAE mapper to meet various JAX-RS exception situations.
The question is if WAE customization is needed. If no - let CXF do it. If yes, how a user can customize WAE processing if what CXF provides OOB does not suit.

(b) would be one option - it is less JAX-RS and more CXF centric solution - I added that flag because at some point CXF were always giving the preference to the user-provided providers - so custom RuntimeException or Exception mappers were able to catch WAE at some point of time - but later on, after some TCK JIRA issues I ended up making the code more spec aligned - this caused some side-effects so I added the flag to keep the user code which depended on the original selection of the mappers operating.

Yes another option is introduced a mapper specifically typed to handle WAE - so you'd have an Exception mapper to handle all but WAE exceptions and a dedicated mapper to handle WAE only - example - it may extend a CXF mapper and customize few things or do a WAE mapping from scratch

Cheers, Sergey




WDYT?
Regards.

[1]
https://github.com/apache/cxf/blob/3.1.x-fixes/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/impl/WebApplicationExceptionMapper.java

[2]
https://github.com/apache/syncope/blob/2_0_X/core/rest-cxf/src/main/java/org/apache/syncope/core/rest/cxf/RestServiceExceptionMapper.java#L102-L110



Reply via email to