Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.3, 3.0, 3.0.1, 3.0.2, 4.0.2, 4.0.1, 4.0, 3.0.5
    • Fix Version/s: 4.1
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      tomcat

      Description

      I hit a major bug with proxies after trying our website with tapestry 4 + friendly URLs. Our ISA server is configured to proxy agressively and as a consequence, tapestry pages got cached although their content is dynamic. After comparing this to another webserver for which the caching did not occur, the other webserver had a "Cache-Control" directive.
      I suggest for tapestry to add a "Cache-Control" header equals to "private,no-cache,no-store" when a page is requested and "max-age=3600" for assets.

        Activity

        Hide
        Henri Dupre added a comment -

        Ok I'll bring the discussion about the default. The disableCaching is great.
        The meta is not enough though to prevent caching on all platforms. ISA Server for instance did not use the meta tag but requires headers on the http response about the cache control. It would be really nice if the parameter on the @Shell could allow to add these http parameters too.
        I had to implement a webrequestservicerfilter to do that.

        Show
        Henri Dupre added a comment - Ok I'll bring the discussion about the default. The disableCaching is great. The meta is not enough though to prevent caching on all platforms. ISA Server for instance did not use the meta tag but requires headers on the http response about the cache control. It would be really nice if the parameter on the @Shell could allow to add these http parameters too. I had to implement a webrequestservicerfilter to do that.
        Hide
        Jesse Kuhnert added a comment -

        I only meant the new "disableCaching" parameter of Shell.

        It's going to be impossible for me to come up with a decision on my own for this one. This ticket is good to have for reference, but I'd suggest bringing this up on either the users/dev mailing list to see what others think.

        Show
        Jesse Kuhnert added a comment - I only meant the new "disableCaching" parameter of Shell. It's going to be impossible for me to come up with a decision on my own for this one. This ticket is good to have for reference, but I'd suggest bringing this up on either the users/dev mailing list to see what others think.
        Hide
        Henri Dupre added a comment -

        I know about the @Shell and I implemented this but I think these would be good headers to have by default.
        Microsoft IIS + .Net do output these by default. I believe several other web apps do that.

        Show
        Henri Dupre added a comment - I know about the @Shell and I implemented this but I think these would be good headers to have by default. Microsoft IIS + .Net do output these by default. I believe several other web apps do that.
        Hide
        Jesse Kuhnert added a comment -

        Check the new @Shell documentation.

        Show
        Jesse Kuhnert added a comment - Check the new @Shell documentation.

          People

          • Assignee:
            Unassigned
            Reporter:
            Henri Dupre
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development