You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cocoon.apache.org by bu...@apache.org on 2003/12/11 12:48:52 UTC

DO NOT REPLY [Bug 25437] New: - [PATCH] TreeProcessor.ForwardEnvironmentWrapper should delegate isResponseModified/setResponseIsNotModified to wrapped environment

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25437>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25437

[PATCH] TreeProcessor.ForwardEnvironmentWrapper should delegate isResponseModified/setResponseIsNotModified to wrapped environment

           Summary: [PATCH] TreeProcessor.ForwardEnvironmentWrapper should
                    delegate isResponseModified/setResponseIsNotModified to
                    wrapped environment
           Product: Cocoon 2
           Version: Current CVS 2.1
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: core
        AssignedTo: dev@cocoon.apache.org
        ReportedBy: m_rolappe@web.de
                CC: m_rolappe@web.de


suppose I have following in the sitemap:

...
    <map:match pattern="">
        <map:redirect-to uri="index.html"/>
    </map:match>

    <map:match pattern="index.html">
        ...
        <map:serialize type="html"/>
    </map:match>
...

even if the response for index.html is completely cacheable and a corresponding
conditional GET request would yield a 304 http status code, this is not true for
the redirecting 'resource'. the reason is that ForwardEnvironmentWrapper doesn't
delegate to the wrapped environment the method calls required for this behavior.

the attached patch adds this delegation to ForwardEnvironmentWrapper.