Caching and headers

[eluser]Arjen van Bochoven[/eluser]
Ok, with CI 1.7.0 the above fix is broken. The format of $this->header array has changed.

I also need header caching, so I used serialize() to store the header data. This will make you less dependent on the internal storage mechanism of the headers. I also separated the expiration string. In practice it looks like this:

in _write_cache() I replaced

Code:

fwrite($fp, $expire.'TS--->'.$output);

with

Code:

fwrite($fp, serialize( array( 'exp' => $expire, 'headers' => $this->headers, 'output' => $output)));

and in _display_cache(), I replaced

Code:

// Strip out the embedded timestamp        
if ( ! preg_match("/(\d+TS--->)/", $cache, $match))
{
    return FALSE;
}

// Has the file expired? If so we'll delete it.
if (time() >= trim(str_replace('TS--->', '', $match['1'])))
{        
    @unlink($filepath);
    log_message('debug', "Cache file has expired. File deleted");
    return FALSE;
}

// Display the cache
$this->_display(str_replace($match['0'], '', $cache));

with

Code:

// Restore the cache array
$cache = unserialize($cache);

// Validate cache file
if ( ! isset($cache['exp'] ) OR ! isset($cache['headers']) OR ! isset($cache['output']))
{
    return FALSE;
}

// Has the file expired? If so we'll delete it.
if (time() >= trim($cache['exp']))
{        
    @unlink($filepath);
    log_message('debug', "Cache file has expired. File deleted");
    return FALSE;
}

// Restore headers
$this->headers = $cache['headers'];
        
// Display the cache
$this->_display($cache['output']);