You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2016/09/16 14:32:20 UTC
[jira] [Commented] (WW-4674) StrutsPrepareAndExecuteFilter should
check for response commited status
[ https://issues.apache.org/jira/browse/WW-4674?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15496467#comment-15496467 ]
ASF subversion and git services commented on WW-4674:
-----------------------------------------------------
Commit 459e198022e8b96e9c3d5f97b1d4c428794ffb96 in struts's branch refs/heads/master from [~lukaszlenart]
[ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=459e198 ]
WW-4674 Uses the same logic as in Dispatcher to control handling exception
> StrutsPrepareAndExecuteFilter should check for response commited status
> ------------------------------------------------------------------------
>
> Key: WW-4674
> URL: https://issues.apache.org/jira/browse/WW-4674
> Project: Struts 2
> Issue Type: Improvement
> Affects Versions: 2.3.30
> Reporter: Mirek Hankus
> Fix For: 2.3.31, 2.5.3
>
>
> In StrutsPrepareAndExecuteFilter in doFilter method there is a code fragment
> {code}
> ActionMapping mapping = this.prepare.findActionMapping(request, response, true);
> if(mapping == null) {
> boolean handled = this.execute.executeStaticResourceRequest(request, response);
> if(!handled) {
> chain.doFilter(request, response);
> }
> } else {
> this.execute.executeAction(request, response, mapping);
> }
> {code}
> Problem is that {{this.prepare.findActionMapping(request, response, true)}} can commit response (in case of exception), but filter continues with execution of chain, in my case causing problems up in the chain.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)