[weld-issues] [JBoss JIRA] Updated: (WELD-865) Weld allows for concurrent call to conversation

[George Sapountzis (JIRA)]

     [ https://issues.jboss.org/browse/WELD-865?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

George Sapountzis updated WELD-865:
-----------------------------------

    Attachment: untested-2.patch


- slightly better naming
- move the non-existent conversation check within conversationContext.activate(cid) where it seems to fit better with the flow

> Weld allows for concurrent call to conversation
> -----------------------------------------------
>
>                 Key: WELD-865
>                 URL: https://issues.jboss.org/browse/WELD-865
>             Project: Weld
>          Issue Type: Bug
>          Components: Scopes & Contexts, Web Tier integration (JSF, JSP, EL and Servlet) 
>    Affects Versions: 1.1.0.Final
>            Reporter: George Sapountzis
>            Assignee: Ales Justin
>             Fix For: 1.2.0.Beta1
>
>         Attachments: untested-2.patch, untested-test-failure-condition-first.patch
>
>
> Concurrent calls to a @ConversationScoped component:
> Thread 1:
> AbstractConversationContext.activate(String cid)
> ConversationImpl.lock(long timeout) <-- returns true
> correctly proceed inside method
> Thread 2:
> AbstractConversationContext.activate(String cid)
> ConversationImpl.lock(long timeout) <-- returns false but activate() does not check result !!!
> *incorrectly* proceed inside method
> ----
> I think AbstractConversationContext.activate(String cid) should check the result of ConversationImpl.lock(long timeout) and not allow the second thread to proceed.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira