Ticket #1823 (new enhancement)

Opened 12 years ago

Last modified 9 years ago

Can't tweak CPSSkins ImageBox and CPSPortlets Image Portlet caching headers

Reported by: madarche Owned by: jmorliaguet
Priority: P3 Milestone: CPS 3.4.10
Component: CPSSkins Version: TRUNK
Severity: normal Keywords: optimization


One uses the CPSSkins ImageBoxes? and Image Portlets to have theme a web site. Those resources are thus heavily used and constrain performances.

Unfortunately those resources are neither influenced by the Caching Policy Manager (located at the CPS root), neither selectable (through the associate tab) for Cache Manager.

Thus one cannot set such thing as RESPONSE.setHeader('Cache-Control', 'max-age=36000') to the request just as we do for Filesystem Images.

Change History

comment:1 Changed 12 years ago by madarche

JMO says it might be related to  http://zope.org/Collectors/CMF/454 but ImageBox? uses OFS.Image.

comment:2 Changed 12 years ago by madarche

  • Keywords optimization added; optmization removed

comment:3 Changed 12 years ago by madarche

  • Priority changed from P2 to P3
  • Type changed from defect to enhancement

To be more precise, there are "Last-Modified" cache headers which result in web browser queries being answered by "304 Not Modified" responses. Caching reverse proxies such as Varnish or Squid can use those informations, thus making this bug less important.

comment:4 Changed 10 years ago by gracinet

  • Summary changed from CPSSkins ImageBox and CPSPortlets Image Portlet produce browser non-cacheable images to Can't tweak CPSSkins ImageBox and CPSPortlets Image Portlet caching headers

By default, Last-Modified is usually been used by user agents and intermediate agents to decide whether to reload: the recommended algorithm in RFC2616 compares this with the date of last request issued by agent and the current date. The older Last-Modified is, the less the agents will issue update (If-Modified-Since) requests.

Moreover, 304 responses are really quick. It is also a good thing not to impose a hard cache time, because this lets users issue force reloads to update a page right away.

comment:5 Changed 9 years ago by gracinet

  • Milestone changed from CPS 3.5.0 to CPS 3.4.10

I should stop repeating myself. The default theme system in trunk is CPSDesignerThemes now

Note: See TracTickets for help on using tickets.