[weld-dev] Follow up to WELD-2611
Benjamin Confino
BENJAMIC at uk.ibm.com
Fri Feb 21 07:46:57 EST 2020
You're welcome
Regards
Benjamin
From: Matej Novotny <manovotn at redhat.com>
To: Benjamin Confino <BENJAMIC at uk.ibm.com>
Cc: Takayuki T Ishii <EBB0F3L at jp.ibm.com>, weld-dev at lists.jboss.org
Date: 21/02/2020 11:58
Subject: [EXTERNAL] Re: [weld-dev] Follow up to WELD-2611
Thanks!
Matej
----- Original Message -----
> From: "Benjamin Confino" <BENJAMIC at uk.ibm.com>
> To: "Matej Novotny" <manovotn at redhat.com>
> Cc: "Takayuki T Ishii" <EBB0F3L at jp.ibm.com>, weld-dev at lists.jboss.org
> Sent: Friday, February 21, 2020 11:35:26 AM
> Subject: RE: [weld-dev] Follow up to WELD-2611
>
> Hello, two PRs coming right up:
>
>
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_weld_core_pull_1967&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=hl8XiFr1UrTSgyXVc4KO_i51sM3Gyhlu1KZ4TkyseEQ&m=0vGWg-1UVoP82OiwVb3Fxpq2DcS-iDP01PRUDct_56I&s=JG_yk6y-v3v-Ix_FWTe7lQi_jYROKRJoiMQILn6vFUU&e=
>
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_weld_core_pull_1966&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=hl8XiFr1UrTSgyXVc4KO_i51sM3Gyhlu1KZ4TkyseEQ&m=0vGWg-1UVoP82OiwVb3Fxpq2DcS-iDP01PRUDct_56I&s=i65phKS4Sk96XzHhT7n50HC3j50Ku9lAvpFahXz1E04&e=
>
> I tested it locally by compiling the class and pasting it into my
liberty
> server, it all looks good.
>
> Regards
> Benjamin
>
>
>
> From: Matej Novotny <manovotn at redhat.com>
> To: Benjamin Confino <BENJAMIC at uk.ibm.com>
> Cc: weld-dev at lists.jboss.org, Takayuki T Ishii <EBB0F3L at jp.ibm.com>
> Date: 21/02/2020 07:52
> Subject: [EXTERNAL] Re: [weld-dev] Follow up to WELD-2611
>
>
>
> Hi,
>
> good catch! Go ahead and send the PRs, I'll reopen that WELD issue and
> link them to it.
> Also, please add a test for this (basically just enhance the one I had
in
>
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_weld_core_pull_1962_files&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=hl8XiFr1UrTSgyXVc4KO_i51sM3Gyhlu1KZ4TkyseEQ&m=W9RpOoqoLA6RSo3b8XcGes3YW9Q_EXHMl0TYhWoriHM&s=LVekwROUfmDvvWmUGB-522FJCLOUS5cFN4p_XbPr-pg&e=
> ).
>
> Matej
>
> ----- Original Message -----
> > From: "Benjamin Confino" <BENJAMIC at uk.ibm.com>
> > To: weld-dev at lists.jboss.org
> > Cc: "Takayuki T Ishii" <EBB0F3L at jp.ibm.com>
> > Sent: Thursday, February 20, 2020 10:20:25 PM
> > Subject: [weld-dev] Follow up to WELD-2611
> >
> > Hello
> >
> > I heard back from my excellent L2 support colleague who has tested the
> weld
> > fix and he says that it works perfect when you append something like
> > "?cid=100000" to the url but not if you append "?cid=" I debugged into
> the
> > code and I believe that what is happening is that:
> >
> > AbstractConversationContext .initilize treats a null or an isEmpty()
cid
> as
> > as a trigger to enter the else block and call
> > associateRequestWithNewConversation()
> >
> > However LazyHttpConversationContextImpl .checkContextInitialized()
only
> calls
> > fireInitEvent() for a null cid or in the catch block for a
> > ConversationNotActiveException.
> >
> > The two methods have differing understandings of what's triggers a new
> > conversation, so the "?cid=" case falls through the gap. I think this
> can be
> > fixed by adding an "|| cid.isEmpty()" to
LazyHttpConversationContextImpl
> > .checkContextInitialized() above the non-catch block call to
> > fireInitEvent(), if you agree I can spin up a couple of pull requests.
> >
> > Regards
> > Benjamin
> > Unless stated otherwise above:
> > IBM United Kingdom Limited - Registered in England and Wales with
number
> > 741598.
> > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6
> 3AU
> >
> > _______________________________________________
> > weld-dev mailing list
> > weld-dev at lists.jboss.org
> >
>
https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.jboss.org_mailman_listinfo_weld-2Ddev&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=hl8XiFr1UrTSgyXVc4KO_i51sM3Gyhlu1KZ4TkyseEQ&m=W9RpOoqoLA6RSo3b8XcGes3YW9Q_EXHMl0TYhWoriHM&s=_VegYIMmqx_N_dSOfjtzDBUbMVVPhgDvJ71mw6fs9fo&e=
>
>
>
>
>
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6
3AU
>
Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number
741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/weld-dev/attachments/20200221/e8309fd7/attachment-0001.html
More information about the weld-dev
mailing list