Re: [2/5] tomee git commit: Added Jackson notice to plus and plume.

2018-10-13 Thread Romain Manni-Bucau
without is what we do @geronimo and what makes sense at MP IMHO since otherwise you just start breaking the consistency of the stack in terms of exposed API (@spec level, not tomee one) and dependencies (tomee level) I know currently the governance in MP is quite bad since each spec lead tries to

Re: [2/5] tomee git commit: Added Jackson notice to plus and plume.

2018-10-13 Thread Roberto Cortez
We do need it to pass the TCK. So what would you recommend? Should we ship without it? Or we can just ship it with the MP flavour. > On 13 Oct 2018, at 22:38, Romain Manni-Bucau wrote: > > It is really optional functionally and throught as such. > It is used to support yaml only - and undertest

Re: [2/5] tomee git commit: Added Jackson notice to plus and plume.

2018-10-13 Thread Romain Manni-Bucau
It is really optional functionally and throught as such. It is used to support yaml only - and undertested to be honest - so normally not used at all in MP apps which are about JSON first (core) and json is there OOTB. Romain Manni-Bucau @rmannibucau | Blog

Re: [2/5] tomee git commit: Added Jackson notice to plus and plume.

2018-10-13 Thread Roberto Cortez
You have an optional dependency to jackson from geronimo open-api. > On 13 Oct 2018, at 22:18, Romain Manni-Bucau wrote: > > wdym? > > Romain Manni-Bucau > @rmannibucau | Blog > | Old Blog >

Re: [2/5] tomee git commit: Added Jackson notice to plus and plume.

2018-10-13 Thread Romain Manni-Bucau
wdym? Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn | Book

Re: [2/5] tomee git commit: Added Jackson notice to plus and plume.

2018-10-13 Thread Roberto Cortez
By the way, and what about openapi? > On 13 Oct 2018, at 22:12, Roberto Cortez wrote: > > Very well. I’ll try to exclude it. > >> On 13 Oct 2018, at 21:48, Romain Manni-Bucau wrote: >> >> PS: this code didn't change since 2 years so we can likely exclude it. Also >> opened https://github.com/

Re: [2/5] tomee git commit: Added Jackson notice to plus and plume.

2018-10-13 Thread Roberto Cortez
Very well. I’ll try to exclude it. > On 13 Oct 2018, at 21:48, Romain Manni-Bucau wrote: > > PS: this code didn't change since 2 years so we can likely exclude it. Also > opened https://github.com/apache/activemq/pull/308 but not sure it will be > that welcomed in current way. > > Romain Manni-

Re: [2/5] tomee git commit: Added Jackson notice to plus and plume.

2018-10-13 Thread Romain Manni-Bucau
PS: this code didn't change since 2 years so we can likely exclude it. Also opened https://github.com/apache/activemq/pull/308 but not sure it will be that welcomed in current way. Romain Manni-Bucau @rmannibucau | Blog | Old B

Re: Suppress CXF Exception

2018-10-13 Thread Ivan Junckes Filho
Thanks a lot for your help Romain. On Sat, Oct 13, 2018 at 4:32 PM Romain Manni-Bucau wrote: > Le sam. 13 oct. 2018 à 19:11, Ivan Junckes Filho a > écrit : > > > Thanks Romain, it works. Is there anyway to make this work without having > > to annotate the client? > > > > i dont think so > > > >

Re: Suppress CXF Exception

2018-10-13 Thread Romain Manni-Bucau
Le sam. 13 oct. 2018 à 19:11, Ivan Junckes Filho a écrit : > Thanks Romain, it works. Is there anyway to make this work without having > to annotate the client? > i dont think so > > Is the property being ignored a bug in mp impl? > Yes and in the client as well AFAIK > > > On Sat, Oct 13,

Re: Suppress CXF Exception

2018-10-13 Thread Ivan Junckes Filho
Thanks Romain, it works. Is there anyway to make this work without having to annotate the client? Is the property being ignored a bug in mp impl? On Sat, Oct 13, 2018 at 1:57 PM Romain Manni-Bucau wrote: > this property is ignored by the MP layer, you have to use a cxf > interceptor: > > @OutI

Re: Suppress CXF Exception

2018-10-13 Thread Romain Manni-Bucau
this property is ignored by the MP layer, you have to use a cxf interceptor: @OutInterceptors(classes = MyInterceptor.class) @Path("foo") public interface MyApi { @GET String get(); } public class MyInterceptor extends AbstractPhaseInterceptor { public MyInterceptor() { super(

Re: Suppress CXF Exception

2018-10-13 Thread Ivan Junckes Filho
I tried this and doesn't seem to work. RestClientBuilder builder = null; builder.property("org.apache.cxf.logging.FaultListener", new NoOpFaultListener()); Is this wrong? On Sat, Oct 13, 2018 at 12:47 PM Romain Manni-Bucau wrote: > Hi Ivan > > you can either configure the log level or set a

Re: Suppress CXF Exception

2018-10-13 Thread Romain Manni-Bucau
Hi Ivan you can either configure the log level or set a FaultListener to handle that and bypass the default log. Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github

Suppress CXF Exception

2018-10-13 Thread Ivan Junckes Filho
Hey guys, I get this when using cxf microprofile impl, is there anyway to suppress this cxf exception, I would like to do proper error handling and I don't want it to be printed on my logs. out 13, 2018 12:18:22 PM org.apache.cxf.phase.PhaseInterceptorChain doDefaultLogging WARNING: Interceptor f