Mailing-List: contact dev-help@cxf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cxf.apache.org
MIME-Version: 1.0
In-Reply-To: <CAGAo6aMxovzMmr5dTmu5t4uLq-pgcefcbrZbpumKua9-JfTeVQ@mail.gmail.com>
References: <CACLE=7P+EM4h2whRBzE_342Kc78FUbHbSiSbHDjsYom0NhQ2wQ@mail.gmail.com>
 <CAOqetn9gb45Lh0i__V2TJGj0p9KWY1N6dYa-+vVmGwz3aX1-bA@mail.gmail.com>
 <CACLE=7OQNnHeUZe=LHQCZL2hPKest4iwRtO+qUZkOpaPjwKtRQ@mail.gmail.com>
 <CAOqetn8HQhjxqSwtLuY_K-PtBB+DHQAMh7wvtAJGhpmK7tHuKA@mail.gmail.com>
 <CACLE=7NYPkeCW1BETB0Tisq35fejT38Xqdxb=1DPh7_hUFO1nA@mail.gmail.com>
 <CAGAo6aMVy29_h+zJKqNnJLPF6BSctcmDYRx1aonR2oCY3cC-Qw@mail.gmail.com>
 <30e2470a-2f14-8d56-8902-a7f4b7b3f18a@gmail.com> <CACLE=7MpseGJz39MV+8k1LDG4zDdS9GgdqvvxkirTAXFMcg4Ew@mail.gmail.com>
 <CAGAo6aNBfc2N-fsiKZOfEzByo+_+bj6rXH=nUS+mQ-b7jfUZ3w@mail.gmail.com>
 <CACLE=7NwtWBEE3pRFWQUKhA83=-efpR7g34OywYfJjqEdfWtrw@mail.gmail.com>
 <CAOqetn8ahVyX2GwX8wPrg4TZPUPjsvdVmNNu16BJNxE-ya0gVA@mail.gmail.com>
 <1666042994.20161018221403@gmail.com> <CACLE=7OZ9NO5nvTspB1zaWH2c3WoD-=Yc95KJk1FAB1ELB3+gQ@mail.gmail.com>
 <CAGAo6aMxovzMmr5dTmu5t4uLq-pgcefcbrZbpumKua9-JfTeVQ@mail.gmail.com>
From: Romain Manni-Bucau <rmannibucau@gmail.com>
Date: Wed, 19 Oct 2016 15:24:29 +0200
Message-ID: <CACLE=7O-JZu0ABPdp_PhwK7zO3FomJuxfZMR9KMHxkbXBF5CoA@mail.gmail.com>
Subject: Re: cxi integration: multiple buses during deployment?
To: Andrey Redko <drreta@gmail.com>
Cc: Daniel Kulp <dev@cxf.apache.org>, "John D. Ament" <johndament@apache.org>
Content-Type: multipart/alternative; boundary=001a11442806d52b5f053f37bb09
archived-at: Wed, 19 Oct 2016 13:25:02 -0000

--001a11442806d52b5f053f37bb09
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

doesn't work but for a weirder reason.

CdiBean is added programmatically - all fine

Then it is directly used to get the bus (in "use the default bus" mode).
This is not really fine since the CDI impl is free to wrap this so you
should retrieve the actual instance of the bus and get it with this Bean
instance and not supposing the CDI container will return you the instance
you added.

Concretely here are the few fixes (on 3.1.x-fixes branch):
https://gist.github.com/rmannibucau/0bb8abf2e164d953be17b24b4a08e685

In CDI you can't assume equals/hashcode of Bean<?> so you need to lookup
the bus instance and not use the internal one. This patch solves the issue.


Romain Manni-Bucau
@rmannibucau <https://twitter.com/rmannibucau> |  Blog
<https://blog-rmannibucau.rhcloud.com> | Old Wordpress Blog
<http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau=
> |
LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber
<http://www.tomitribe.com> | JavaEE Factory
<https://javaeefactory-rmannibucau.rhcloud.com>

2016-10-19 13:35 GMT+02:00 Andrey Redko <drreta@gmail.com>:

> Hi Roman,
>
> Thanks a lot for clarifying on things. Could you please try latest
> 3.1.9-SNAPSHOT / 3.2.0-SNAPSHOT against this issue?
> The fix should be there. Thanks!
>
> Best Regards,
>     Andriy Redko
>
> On Wed, Oct 19, 2016 at 2:28 AM, Romain Manni-Bucau <rmannibucau@gmail.co=
m
> > wrote:
>
>> Le 19 oct. 2016 04:14, "Andriy Redko" <drreta@gmail.com> a =C3=A9crit :
>> >
>> > I think this is what is happening:
>> >  - Roman defines own CXF bus bean, named "cxf"
>>
>> This is a workaround to be able to lookup the bus in my workaround
>> extension. Not doing it without my extension leads to the issue.
>>
>> >  - our JAX-RS CDI extension finds it and makes it a "default" bus to b=
e
>> used
>> >    for JAXRSServerFactoryBean instances
>> >  - however, JAXRSServerFactoryBean::setApplication() would create a
>> new bus
>> >    nonetheless, because the actual call to
>> JAXRSServerFactoryBean::setBus would
>> >    be done later
>> >
>>
>> This is the issue, custom bus or not.
>>
>> > The issue in this case is that application was created using one bus
>> instance,
>> > but JAXRSServerFactoryBean would be initialized using another bus
>> instance. This
>> > is my understading of the problem. Hopefully it makes sense.
>> >
>>
>> The factory uses the cdi bus but the application used a "prototype" bus.
>>
>> Side note: i deploy with tomcat embedded a single app so I am not in
>> "multiple" deployments case.
>>
>> > Best Regards,
>> >     Andriy Redko
>> >
>> > JDA> So still something isn't clicking as I don't have Romain's issue.
>> Is it
>> > JDA> specific to when you have multiple deployments in your JVM?
>> >
>> > JDA> On Oct 18, 2016 09:02, "Romain Manni-Bucau" <rmannibucau@gmail.co=
m>
>> wrote:
>> >
>> > >> 2016-10-18 14:43 GMT+02:00 Andrey Redko <drreta@gmail.com>:
>> >
>> > >> > Hi Romain Manni-Bucau,
>> > >> >
>> > >> > Yes, technically we do have this option, but I would suggest to u=
se
>> > >> > explicit wirings instead of thread local context.  It
>> > >> > could have quite a number of side effects. I hope you would agree
>> with
>> > >> > that.
>> > >> >
>> > >> >
>> > >> I agree it is better but side effect should be null since the value
>> is
>> > >> cleaned up after. If not then runtime can have side effects which
>> would be
>> > >> a bigger issue ;)
>> >
>> >
>> > >> > Thanks!
>> > >> >
>> > >> > Best Regards,
>> > >> >     Andriy Redko
>> > >> >
>> > >> >
>> > >> > On Tue, Oct 18, 2016 at 7:49 AM, Romain Manni-Bucau <
>> > >> rmannibucau@gmail.com
>> > >> > >
>> > >> > wrote:
>> > >> >
>> > >> > > We also have option 3: set the thread bus as in the workaround =
I
>> used.
>> > >> > >
>> > >> > >
>> > >> > > Romain Manni-Bucau
>> > >> > > @rmannibucau <https://twitter.com/rmannibucau> |  Blog
>> > >> > > <https://blog-rmannibucau.rhcloud.com> | Old Wordpress Blog
>> > >> > > <http://rmannibucau.wordpress.com> | Github <https://github.com=
/
>> > >> > > rmannibucau> |
>> > >> > > LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber
>> > >> > > <http://www.tomitribe.com> | JavaEE Factory
>> > >> > > <https://javaeefactory-rmannibucau.rhcloud.com>
>> > >> > >
>> > >> > > 2016-10-18 13:47 GMT+02:00 Sergey Beryozkin <
>> sberyozkin@gmail.com>:
>> > >> > >
>> > >> > > > Hi Andriy
>> > >> > > > yes, option 1 is probably the simplest for the CDI integratio=
n
>> path,
>> > >> > > >
>> > >> > > > Thanks, Sergey
>> > >> > > >
>> > >> > > > On 18/10/16 12:43, Andrey Redko wrote:
>> > >> > > >
>> > >> > > >> Hey guys,
>> > >> > > >>
>> > >> > > >> Indeed, there is an issue with CDI and usage of custom bus. =
I
>> think
>> > >> > > there
>> > >> > > >> are couple of the options
>> > >> > > >> we have here:
>> > >> > > >>  - pass the bus instance to ResourceUtils::createApplication=
,
>> in
>> > >> this
>> > >> > > >> case
>> > >> > > >> if bus is not set, the behaviour would be exactly the same
>> > >> > > >>  - change the way JAXRSServerFactoryBean is being created so
>> we
>> > >> could
>> > >> > > set
>> > >> > > >> the bus before setting the application
>> > >> > > >>
>> > >> > > >> Romain Manni-Bucau, do we have a ticket for it? If not, coul=
d
>> you
>> > >> > please
>> > >> > > >> create one, we'll work on a fix for it.
>> > >> > > >> Thanks.
>> > >> > > >>
>> > >> > > >>
>> > >> > > >> On Tue, Oct 18, 2016 at 7:08 AM, Romain Manni-Bucau <
>> > >> > > >> rmannibucau@gmail.com>
>> > >> > > >> wrote:
>> > >> > > >>
>> > >> > > >> Not yet something failling i can share but globally I ended
>> up doing
>> > >> > > that:
>> > >> > > >>>
>> > >> > > >>> @Named("cxf")
>> > >> > > >>> @ApplicationScoped
>> > >> > > >>> public class BusInstance implements Bus {
>> > >> > > >>> @Delegate
>> > >> > > >>> private Bus delegate =3D new ExtensionManagerBus();
>> > >> > > >>> }
>> > >> > > >>> public class OWBAutoSetup implements
>> ServletContainerInitializer {
>> > >> > > >>> @Override
>> > >> > > >>> public void onStartup(final Set<Class<?>> c, final
>> ServletContext
>> > >> > ctx)
>> > >> > > >>> throws ServletException {
>> > >> > > >>>
>> > >> > > >>> ctx.addListener(WebBeansConfigurationListener.class);
>> > >> > > >>> ctx.addListener(WebBeansConfigurationHttpSessionListener.
>> class);
>> > >> > > >>> }
>> > >> > > >>> }
>> > >> > > >>>
>> > >> > > >>>
>> > >> > > >>>
>> > >> > > >>> public class CxfCdiAutoSetup implements
>> > >> ServletContainerInitializer {
>> > >> > > >>> @Override
>> > >> > > >>> public void onStartup(final Set<Class<?>> c, final
>> ServletContext
>> > >> > ctx)
>> > >> > > >>> throws ServletException {
>> > >> > > >>> final ServletRegistration.Dynamic jaxrs =3D
>> ctx.addServlet("cxf-cdi",
>> > >> > > >>> CXFCdiServlet.class);
>> > >> > > >>>
>> > >> > > >>>
>> > >> > > >>> jaxrs.setLoadOnStartup(1);
>> > >> > > >>> jaxrs.setAsyncSupported(true);
>> > >> > > >>> jaxrs.addMapping("/*"); // TODO: config
>> > >> > > >>> }
>> > >> > > >>> }
>> > >> > > >>>
>> > >> > > >>>
>> > >> > > >>>
>> > >> > > >>> With my workaround extension it works fine, without there i=
s
>> this
>> > >> > > >>> mismatch
>> > >> > > >>> of bus.
>> > >> > > >>>
>> > >> > > >>> Have to admit I have no idea how
>> > >> > > >>> org.apache.cxf.jaxrs.utils.ResourceUtils#createApplication(
>> > >> > javax.ws.rs
>> > >> > > .
>> > >> > > >>> core.Application,
>> > >> > > >>> boolean, boolean) could reuse the right bus in current stat=
e
>> of the
>> > >> > cdi
>> > >> > > >>> module. If you check
>> > >> > > >>> org.apache.cxf.cdi.JAXRSCdiResourceExtension#createFactoryI=
n
>> > >> > > >>> stance(javax.
>> > >> > > >>> ws.rs.core.Application,
>> > >> > > >>> java.util.List<?>, java.util.List<?>, java.util.List<?
>> extends
>> > >> > > >>> org.apache.cxf.feature.Feature>) the bus is set but too lat=
e
>> > >> (after
>> > >> > > bean
>> > >> > > >>> .setApplication(app);).
>> > >> > > >>>
>> > >> > > >>> Romain Manni-Bucau
>> > >> > > >>> @rmannibucau <https://twitter.com/rmannibucau> |  Blog
>> > >> > > >>> <https://blog-rmannibucau.rhcloud.com> | Old Wordpress Blog
>> > >> > > >>> <http://rmannibucau.wordpress.com> | Github <
>> https://github.com/
>> > >> > > >>> rmannibucau> |
>> > >> > > >>> LinkedIn <https://www.linkedin.com/in/rmannibucau> |
>> Tomitriber
>> > >> > > >>> <http://www.tomitribe.com> | JavaEE Factory
>> > >> > > >>> <https://javaeefactory-rmannibucau.rhcloud.com>
>> > >> > > >>>
>> > >> > > >>> 2016-10-18 12:50 GMT+02:00 John D. Ament <
>> johndament@apache.org>:
>> > >> > > >>>
>> > >> > > >>> Do you have a sample project that demonstrates the issue?
>> > >> > > >>>>
>> > >> > > >>>> Also, do you see setBus method being called on
>> CXFCdiServlet?  How
>> > >> > > many
>> > >> > > >>>> times do you see it invoked?  Maybe step through loadBus t=
o
>> see
>> > >> > which
>> > >> > > >>>>
>> > >> > > >>> path
>> > >> > > >>>
>> > >> > > >>>> it follows.
>> > >> > > >>>>
>> > >> > > >>>> On Tue, Oct 18, 2016 at 6:24 AM Romain Manni-Bucau <
>> > >> > > >>>>
>> > >> > > >>> rmannibucau@gmail.com>
>> > >> > > >>>
>> > >> > > >>>> wrote:
>> > >> > > >>>>
>> > >> > > >>>> Not really,
>> > >> > > >>>>>
>> > >> > > >>>>> I don't have any bus bean, just using the default one of
>> the
>> > >> > > extenson -
>> > >> > > >>>>> behavior is the same however if I impl my own bus bean.
>> > >> > > >>>>>
>> > >> > > >>>>> Issue is in the load() method of the extension. It gets
>> the bus
>> > >> and
>> > >> > > >>>>>
>> > >> > > >>>> then
>> > >> > > >>>
>> > >> > > >>>> calls ResourceUtils.createApplication(application, false,
>> false)
>> > >> > > which
>> > >> > > >>>>>
>> > >> > > >>>> will
>> > >> > > >>>>
>> > >> > > >>>>> call org.apache.cxf.jaxrs.JAXRSServ
>> erFactoryBean#setApplication
>> > >> > > which
>> > >> > > >>>>>
>> > >> > > >>>> will
>> > >> > > >>>>
>> > >> > > >>>>> do getBus() wich uses BusFactory.getThreadDefaultBus()
>> which is
>> > >> > > >>>>>
>> > >> > > >>>> obviously
>> > >> > > >>>>
>> > >> > > >>>>> not set so you end up in org.apache.cxf.BusFactory#
>> > >> createThreadBus
>> > >> > > >>>>>
>> > >> > > >>>> which
>> > >> > > >>>
>> > >> > > >>>> does a newInstance().createBus() and here you are, you hav=
e
>> 2
>> > >> > > instances
>> > >> > > >>>>>
>> > >> > > >>>> of
>> > >> > > >>>>
>> > >> > > >>>>> a bus.
>> > >> > > >>>>>
>> > >> > > >>>>> Romain
>> > >> > > >>>>>
>> > >> > > >>>>> 2016-10-18 12:10 GMT+02:00 John D. Ament <
>> john.d.ament@gmail.com
>> > >> >:
>> > >> > > >>>>>
>> > >> > > >>>>> Romain,
>> > >> > > >>>>>>
>> > >> > > >>>>>> Depends on how you're trying to instantiate it.  There i=
s
>> a
>> > >> > > >>>>>>
>> > >> > > >>>>> CdiBusBean
>> > >> > > >>>
>> > >> > > >>>> provided by CXF which does what you're trying to do -
>> > >> > > >>>>>> https://github.com/apache/cxf/
>> blob/3.1.x-fixes/integration/
>> > >> > > >>>>>> cdi/src/main/java/org/apache/cxf/cdi/CdiBusBean.java#L40
>> > >> > > >>>>>> ,
>> > >> > > >>>>>> take a look at the create method.
>> > >> > > >>>>>>
>> > >> > > >>>>>> I was actually contemplating removing the @Inject from t=
he
>> > >> > > >>>>>>
>> > >> > > >>>>> CXFCdiServlet's
>> > >> > > >>>>>
>> > >> > > >>>>>> set method, it seems to work inconsistently.  However, i
>> suspect
>> > >> > > that
>> > >> > > >>>>>>
>> > >> > > >>>>> your
>> > >> > > >>>>>
>> > >> > > >>>>>> issue is that you're getting a Bus registered as a valid
>> CDI
>> > >> bean
>> > >> > > >>>>>> (bean-discovery-mode=3Dall?).
>> > >> > > >>>>>>
>> > >> > > >>>>>> John
>> > >> > > >>>>>>
>> > >> > > >>>>>> On Tue, Oct 18, 2016 at 5:58 AM Romain Manni-Bucau <
>> > >> > > >>>>>>
>> > >> > > >>>>> rmannibucau@gmail.com>
>> > >> > > >>>>>
>> > >> > > >>>>>> wrote:
>> > >> > > >>>>>>
>> > >> > > >>>>>> Hi guys,
>> > >> > > >>>>>>>
>> > >> > > >>>>>>> in cdi-integration I don't get how the deployment can
>> work
>> > >> cause
>> > >> > > >>>>>>>
>> > >> > > >>>>>> the
>> > >> > > >>>
>> > >> > > >>>> thread
>> > >> > > >>>>>>
>> > >> > > >>>>>>> local bus is not set
>> > >> > > >>>>>>>
>> > >> > > >>>>>>> Here what i did to ensure i use a single bus (and
>> prevented the
>> > >> > cxf
>> > >> > > >>>>>>>
>> > >> > > >>>>>> one
>> > >> > > >>>>
>> > >> > > >>>>> to
>> > >> > > >>>>>>
>> > >> > > >>>>>>> run):
>> > >> > > >>>>>>>
>> > >> > > >>>>>>> public class JAXRSCdiResourceExtensionWorkaround extend=
s
>> > >> > > >>>>>>> JAXRSCdiResourceExtension {
>> > >> > > >>>>>>>     @Override
>> > >> > > >>>>>>>     public void load(@Observes final
>> AfterDeploymentValidation
>> > >> > > >>>>>>>
>> > >> > > >>>>>> event,
>> > >> > > >>>
>> > >> > > >>>> final BeanManager beanManager) {
>> > >> > > >>>>>>>         final Bus bus =3D
>> > >> > > >>>>>>>
>> > >> > > >>>>>>> Bus.class.cast(beanManager.getReference(beanManager.
>> > >> > > >>>>>>>
>> > >> > > >>>>>> resolve(beanManager.getBeans(Bus.class)),
>> > >> > > >>>>>>
>> > >> > > >>>>>>> Bus.class, null));
>> > >> > > >>>>>>>         BusFactory.setThreadDefaultBus(bus); // cause
>> app
>> > >> class
>> > >> > > >>>>>>>
>> > >> > > >>>>>> will
>> > >> > > >>>
>> > >> > > >>>> rely on that and would create multiple bus and then
>> deployment
>> > >> > > >>>>>>>
>> > >> > > >>>>>> would
>> > >> > > >>>
>> > >> > > >>>> be broken
>> > >> > > >>>>>>>         try {
>> > >> > > >>>>>>>             super.load(event, beanManager);
>> > >> > > >>>>>>>         } finally {
>> > >> > > >>>>>>>             BusFactory.clearDefaultBusForAnyThread(bus)=
;
>> > >> > > >>>>>>>         }
>> > >> > > >>>>>>>     }
>> > >> > > >>>>>>> }
>> > >> > > >>>>>>>
>> > >> > > >>>>>>>
>> > >> > > >>>>>>> Issue was caused by JAXRSCdiResourceExtension#
>> > >> > > >>>>>>>
>> > >> > > >>>>>> createFactoryInstance
>> > >> > > >>>
>> > >> > > >>>> which
>> > >> > > >>>>>>
>> > >> > > >>>>>>> calls ResourceUtils.createApplication which uses the
>> thread
>> > >> bus
>> > >> > > >>>>>>>
>> > >> > > >>>>>> which
>> > >> > > >>>>
>> > >> > > >>>>> is
>> > >> > > >>>>>
>> > >> > > >>>>>> not set by the extension leading to 2 buses.
>> > >> > > >>>>>>>
>> > >> > > >>>>>>> Did I miss something?
>> > >> > > >>>>>>>
>> > >> > > >>>>>>> Romain Manni-Bucau
>> > >> > > >>>>>>> @rmannibucau <https://twitter.com/rmannibucau> |  Blog
>> > >> > > >>>>>>> <https://blog-rmannibucau.rhcloud.com> | Old Wordpress
>> Blog
>> > >> > > >>>>>>> <http://rmannibucau.wordpress.com> | Github <
>> > >> > > >>>>>>> https://github.com/rmannibucau> |
>> > >> > > >>>>>>> LinkedIn <https://www.linkedin.com/in/rmannibucau> |
>> > >> Tomitriber
>> > >> > > >>>>>>> <http://www.tomitribe.com> | JavaEE Factory
>> > >> > > >>>>>>> <https://javaeefactory-rmannibucau.rhcloud.com>
>> > >> > > >>>>>>>
>> > >> > > >>>>>>>
>> > >> > > >>>>>>
>> > >> > > >>>>>
>> > >> > > >>>>
>> > >> > > >>>
>> > >> > > >>
>> > >> > > >
>> > >> > > > --
>> > >> > > > Sergey Beryozkin
>> > >> > > >
>> > >> > > > Talend Community Coders
>> > >> > > > http://coders.talend.com/
>> > >> > > >
>> > >> > >
>> > >> >
>> >
>> >
>>
>
>

--001a11442806d52b5f053f37bb09--