Yep, or we just do it OOTB for the MP distro in a first step. I don't have any strong opinion since in all cases we'll break some users :(.
Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book <https://www.packtpub.com/application-development/java-ee-8-high-performance> Le jeu. 2 août 2018 à 17:26, Roberto Cortez <radcor...@yahoo.com.invalid> a écrit : > I understand. > I think we need to do it, since I've found a couple of issues with the MP > TCK using models with Jsonb annotations that were not being applied due to > the missing provider. And Jsonb is also part of EE 8, so I believe this > should be the default behaviour. > To return to the old behaviour, we could have instructions to setup the > old provider via system.properties, right? Via cxf.jaxrs.providers? > On Thursday, August 2, 2018, 3:47:48 PM GMT+1, Romain Manni-Bucau < > rmannibu...@gmail.com> wrote: > > Hi Roberto, > > You can't get this one and the old johnzon mapper at the same time (both > will conflict). > I'm all for migrating to jsonb but note it will break end users of TomEE 7 > so we should IMHO ensure the way to configure back to the old behavior > globally (without modifying the app) is well documented on the jaxrs mapper > page. > > Romain Manni-Bucau > @rmannibucau <https://twitter.com/rmannibucau> | Blog > <https://rmannibucau.metawerx.net/> | Old Blog > <http://rmannibucau.wordpress.com> | Github < > https://github.com/rmannibucau> | > LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book > < > https://www.packtpub.com/application-development/java-ee-8-high-performance > > > > > Le jeu. 2 août 2018 à 16:16, Roberto Cortez <radcor...@yahoo.com.invalid> > a > écrit : > > > Hi, > > I was wondering if it will be ok to add the Johnzon Jsonb Provider to the > > default providers list? The dependency is already in the project, it was > > just commented out with a "java 8 only". I guess this was pre TomEE 8. > > If this is ok, here is a PR that uncomments the dependency and add the > > org.apache.johnzon.jaxrs.jsonb.jaxrs.JsonbJaxrsProvider in the providers > > list. > > Cheers,Roberto
