Skip to content
Snippets Groups Projects
Commit 34910e36 authored by Demian Katz's avatar Demian Katz
Browse files

Merge pull request #161 from zsrlibrary/browser_cacheable_cover_images

Set proper headers on Cover Images so they can be cached by the browser
parents b2f5cceb e75730d5
Branches
Tags
No related merge requests found
...@@ -109,6 +109,22 @@ class CoverController extends AbstractBase ...@@ -109,6 +109,22 @@ class CoverController extends AbstractBase
$headers->addHeaderLine( $headers->addHeaderLine(
'Content-type', $this->getLoader()->getContentType() 'Content-type', $this->getLoader()->getContentType()
); );
// Send proper caching headers so that the user's browser
// is able to cache the cover images and not have to re-request
// then on each page load. Default TTL set at 14 days
$coverImageTtl = (60*60*24*14); // 14 days
$headers->addHeaderLine(
'Cache-Control', "maxage=".$coverImageTtl
);
$headers->addHeaderLine(
'Pragma', 'public'
);
$headers->addHeaderLine(
'Expires', gmdate('D, d M Y H:i:s', time()+$coverImageTtl) . ' GMT'
);
$response->setContent($this->getLoader()->getImage()); $response->setContent($this->getLoader()->getImage());
return $response; return $response;
} }
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment