I see, maybe something doesn't hurting much CXF would be to add a flag in the primitive provider. This way users could at least configure it to go back to the previous behavior.
Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber <http://www.tomitribe.com> 2015-05-06 10:54 GMT+02:00 Sergey Beryozkin <sberyoz...@gmail.com>: > Hi Romain > > Looks like you have a very nice set of tests which is impressive :-) > > That was a 'side-effect' of getting a spec requirement enforced. It is > about section 4.2.6, > "java.lang.Boolean , java.lang.Character , java.lang.Number Only for > text/plain" > > Personally I think it is too restrictive but a TCK test is failing against > CXF right now, which is partially related to this requirement, hence this > restriction needs to be enforced first for the final test solution to be > done. > Now that it is done it would be sensitive still continue to support > Boolean/etc for any media type by default - we've had some troubles with > the matching algorithm offering a more flexible sub-resource selection in > some of the edge cases by default earlier... > > Thanks, Sergey > > > On 06/05/15 09:23, Romain Manni-Bucau wrote: > >> Hi guys, >> >> org.apache.cxf.jaxrs.provider.PrimitiveTextProvider#isSupported changed >> when upgrading to also validate the media type. >> >> This sounds better but it is actually a regression since before it was >> supported with any media type. >> >> A solution could be to have it twice in the writer chain, once with this >> restriction at the beginning and another time without this restriction at >> the end as a fallback. >> >> Not sure it does worth a fix but was a small side effect I experienced >> upgrading. >> >> Romain Manni-Bucau >> @rmannibucau <https://twitter.com/rmannibucau> | Blog >> <http://rmannibucau.wordpress.com> | Github < >> https://github.com/rmannibucau> | >> LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber >> <http://www.tomitribe.com> >> >> >
