Hello, two PRs coming right up: https://github.com/weld/core/pull/1967 https://github.com/weld/core/pull/1966
I tested it locally by compiling the class and pasting it into my liberty server, it all looks good. Regards Benjamin From: Matej Novotny <[email protected]> To: Benjamin Confino <[email protected]> Cc: [email protected], Takayuki T Ishii <[email protected]> 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" <[email protected]> > To: [email protected] > Cc: "Takayuki T Ishii" <[email protected]> > 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 > [email protected] > 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
_______________________________________________ weld-dev mailing list [email protected] https://lists.jboss.org/mailman/listinfo/weld-dev
