'Last-Modified' and 'Content-Length' HTTP headers added for better image caching

This commit is contained in:
Max Kostikov 2018-11-06 13:37:50 +01:00
parent 873fa16517
commit e116b90116

View File

@ -33,8 +33,6 @@ class Photo extends \Zotlabs\Web\Controller {
$observer_xchan = get_observer_hash(); $observer_xchan = get_observer_hash();
$default = z_root() . '/' . get_default_profile_photo();
if(isset($type)) { if(isset($type)) {
/** /**
@ -44,22 +42,24 @@ class Photo extends \Zotlabs\Web\Controller {
if($type === 'profile') { if($type === 'profile') {
switch($res) { switch($res) {
case 'm': case 'm':
$resolution = 5; $resolution = 5;
$default = z_root() . '/' . get_default_profile_photo(80); $default = get_default_profile_photo(80);
break; break;
case 's': case 's':
$resolution = 6; $resolution = 6;
$default = z_root() . '/' . get_default_profile_photo(48); $default = get_default_profile_photo(48);
break; break;
case 'l': case 'l':
default: default:
$resolution = 4; $resolution = 4;
$default = get_default_profile_photo();
break; break;
} }
} }
$modified = filemtime($default);
$default = z_root() . '/' . $default;
$uid = $person; $uid = $person;
$d = [ 'imgscale' => $resolution, 'channel_id' => $uid, 'default' => $default, 'data' => '', 'mimetype' => '' ]; $d = [ 'imgscale' => $resolution, 'channel_id' => $uid, 'default' => $default, 'data' => '', 'mimetype' => '' ];
@ -78,11 +78,13 @@ class Photo extends \Zotlabs\Web\Controller {
intval(PHOTO_PROFILE) intval(PHOTO_PROFILE)
); );
if($r) { if($r) {
$modified = strtotime($r[0]['edited']);
$data = dbunescbin($r[0]['content']); $data = dbunescbin($r[0]['content']);
$mimetype = $r[0]['mimetype']; $mimetype = $r[0]['mimetype'];
} }
if(intval($r[0]['os_storage'])) if(intval($r[0]['os_storage'])) {
$data = file_get_contents($data); $data = file_get_contents($data);
}
} }
if(! $data) { if(! $data) {
$data = fetch_image_from_url($default,$mimetype); $data = fetch_image_from_url($default,$mimetype);
@ -164,6 +166,7 @@ class Photo extends \Zotlabs\Web\Controller {
if($exists && $allowed) { if($exists && $allowed) {
$data = dbunescbin($e[0]['content']); $data = dbunescbin($e[0]['content']);
$mimetype = $e[0]['mimetype']; $mimetype = $e[0]['mimetype'];
$modified = strtotime($e[0]['edited']);
if(intval($e[0]['os_storage'])) { if(intval($e[0]['os_storage'])) {
$streaming = $data; $streaming = $data;
} }
@ -183,7 +186,6 @@ class Photo extends \Zotlabs\Web\Controller {
if(! isset($data)) { if(! isset($data)) {
if(isset($resolution)) { if(isset($resolution)) {
switch($resolution) { switch($resolution) {
case 4: case 4:
$data = fetch_image_from_url(z_root() . '/' . get_default_profile_photo(),$mimetype); $data = fetch_image_from_url(z_root() . '/' . get_default_profile_photo(),$mimetype);
break; break;
@ -210,10 +212,14 @@ class Photo extends \Zotlabs\Web\Controller {
} }
} }
// @FIXME Seems never invoked
// Writing in cachefile // Writing in cachefile
if (isset($cachefile) && $cachefile != '') if (isset($cachefile) && $cachefile != '') {
file_put_contents($cachefile, $data); file_put_contents($cachefile, $data);
$modified = filemtime($cachefile);
}
if(function_exists('header_remove')) { if(function_exists('header_remove')) {
header_remove('Pragma'); header_remove('Pragma');
header_remove('pragma'); header_remove('pragma');
@ -247,6 +253,7 @@ class Photo extends \Zotlabs\Web\Controller {
} }
// If it's a file resource, stream it. // If it's a file resource, stream it.
if($streaming && $channel) { if($streaming && $channel) {
@ -262,6 +269,8 @@ class Photo extends \Zotlabs\Web\Controller {
} }
} }
else { else {
header("Last-Modified: " . gmdate("D, d M Y H:i:s", $modified) . " GMT");
header("Content-Length: " . strlen($data));
echo $data; echo $data;
} }