[ 
https://jira.nuxeo.com/browse/NXP-7823?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=105765#comment-105765
 ] 

Florent Guillaume commented on NXP-7823:
----------------------------------------

Doing buffering now in NuxeoRequestControllerFilter and WebEngineFilter.
http://hg.nuxeo.org/nuxeo/nuxeo-services/rev/c4870f3bebd7
http://hg.nuxeo.org/nuxeo/nuxeo-webengine/rev/c1f3e060503d
http://hg.nuxeo.org/nuxeo/nuxeo-theme/rev/5d5bfc955acf


> Always commit the transaction before sending the response
> ---------------------------------------------------------
>
>                 Key: NXP-7823
>                 URL: https://jira.nuxeo.com/browse/NXP-7823
>             Project: Nuxeo Enterprise Platform
>          Issue Type: Bug
>          Components: Web Foundations
>    Affects Versions: 5.3.2
>            Reporter: Florent Guillaume
>            Assignee: Florent Guillaume
>            Priority: Critical
>             Fix For: 5.5
>
>
> When transactions are activated, {{NuxeoRequestControllerFilter}} wraps the 
> response in a {{CommittingServletResponseWrapper}} to ensure that a commit is 
> done before any byte is sent to the client.
> But this means that the rest of the server processing will be done without a 
> transaction present, which could be worse than the original solution for the 
> problem (NXP-5129). The problem is that if the rest of the code runs without 
> a transaction, any session opening for instance will not get a session whose 
> invalidations have been processed so will potentially see invalid state.
> Instead should buffer the whole response, then commit in 
> {{NuxeoRequestControllerFilter}}, then send the response. There should be a 
> way to bypass this for processes that really need to stream a response to the 
> client (file download).

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

        
_______________________________________________
ECM-tickets mailing list
ECM-tickets@lists.nuxeo.com
http://lists.nuxeo.com/mailman/listinfo/ecm-tickets

Reply via email to