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>
