[Tickets #9315] Services cache is not issuing the correct headers

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/9315
------------------------------------------------------------------------------
  Ticket             | 9315
  Created By         | goncalo.queiros at portugalmail.net
  Summary            | Services cache is not issuing the correct headers
  Queue              | Horde Base
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


goncalo.queiros at portugalmail.net (2010-10-18 09:33) wrote:

Currently, files (javascript and css) served by services/cache.php are  
not getting the proper header, making them not cache-able by the  
browser.

The problem is that cache.php issues a Horde_Registry::appInit, that  
will then call the Registry constructor which in turn calls the  
Horde_Session construct, where the session is started and the headers  
are issued.
The problem is that the Horde_Session constructor is relying on the  
existence of the $registry->initParams['session_cache_limiter']  
variable:

session_cache_limiter(is_null($registry->initParams['session_cache_limiter'])  
? $conf['session']['cache_limiter'] :  
$registry->initParams['session_cache_limiter']);

But this params are only setted before the Horde_Registry is called,  
thus the session_cache_limiter will only be called with the $conf  
variable which value is 'nocache'.

I did solve the problem, but i don't think this is the intended way,  
but i attach the patch anyway