From 57ebd7e1704597b1eea31610cd08c076ea7473cf Mon Sep 17 00:00:00 2001 From: Max Kostikov Date: Mon, 3 Dec 2018 13:10:55 +0100 Subject: New logic for less dependence on cache logic --- Zotlabs/Module/Photo.php | 27 ++++++++++++--------------- 1 file changed, 12 insertions(+), 15 deletions(-) (limited to 'Zotlabs/Module') diff --git a/Zotlabs/Module/Photo.php b/Zotlabs/Module/Photo.php index d85db7307..da5d4c39b 100644 --- a/Zotlabs/Module/Photo.php +++ b/Zotlabs/Module/Photo.php @@ -139,7 +139,7 @@ class Photo extends \Zotlabs\Web\Controller { $resolution = 1; } - $r = q("SELECT uid, photo_usage, expires, display_path FROM photo WHERE resource_id = '%s' AND imgscale = %d LIMIT 1", + $r = q("SELECT uid, photo_usage, display_path FROM photo WHERE resource_id = '%s' AND imgscale = %d LIMIT 1", dbesc($photo), intval($resolution) ); @@ -156,20 +156,17 @@ class Photo extends \Zotlabs\Web\Controller { if(! in_array($resolution,[4,5,6])) $allowed = (-1); if($u === PHOTO_CACHE) { - // Cached image leak protection - if(! (local_channel() || $cache_mode['leak'])) { - header("Location: " . $r[0]['display_path']); - killme(); - } - // Revalidate cache - if($cache_mode['on'] && strtotime($r[0]['expires']) - 60 < time()) { - $cache = array( - 'url' => $r[0]['display_path'], - 'uid' => $r[0]['uid'] - ); + // Validate cache + $cache = array( + 'resid' => $photo, + 'uid' => $r[0]['uid'], + 'status' => true + ); + if($cache_mode['on']) call_hooks('cache_url_hook', $cache); - if(! $cache['status']) - http_status_exit(404,'not found'); + if((! (local_channel() || $cache_mode['leak'])) || (! $cache['status'])) { + header("Location: " . htmlspecialchars_decode($r[0]['display_path'])); + killme(); } } } @@ -184,7 +181,7 @@ class Photo extends \Zotlabs\Web\Controller { dbesc($photo), intval($resolution) ); - + $exists = (($e) ? true : false); if($exists && $allowed) { -- cgit v1.2.3 From 244c0d828a31447cb215251d88b0123508f0101d Mon Sep 17 00:00:00 2001 From: Max Kostikov Date: Mon, 3 Dec 2018 13:29:27 +0100 Subject: Update Photo.php --- Zotlabs/Module/Photo.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'Zotlabs/Module') diff --git a/Zotlabs/Module/Photo.php b/Zotlabs/Module/Photo.php index da5d4c39b..f96d478f2 100644 --- a/Zotlabs/Module/Photo.php +++ b/Zotlabs/Module/Photo.php @@ -160,7 +160,7 @@ class Photo extends \Zotlabs\Web\Controller { $cache = array( 'resid' => $photo, 'uid' => $r[0]['uid'], - 'status' => true + 'status' => false ); if($cache_mode['on']) call_hooks('cache_url_hook', $cache); -- cgit v1.2.3 From b586fcebabfad0c062deb3070d1679414595dc95 Mon Sep 17 00:00:00 2001 From: Max Kostikov Date: Mon, 3 Dec 2018 21:21:23 +0100 Subject: Update Photo.php --- Zotlabs/Module/Photo.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'Zotlabs/Module') diff --git a/Zotlabs/Module/Photo.php b/Zotlabs/Module/Photo.php index f96d478f2..a73492ae2 100644 --- a/Zotlabs/Module/Photo.php +++ b/Zotlabs/Module/Photo.php @@ -164,7 +164,7 @@ class Photo extends \Zotlabs\Web\Controller { ); if($cache_mode['on']) call_hooks('cache_url_hook', $cache); - if((! (local_channel() || $cache_mode['leak'])) || (! $cache['status'])) { + if(! $cache['status'] && $cache_mode['leak']) { header("Location: " . htmlspecialchars_decode($r[0]['display_path'])); killme(); } -- cgit v1.2.3 From 1c44912aff2c7f026fe1e5e74eb1d70920d17747 Mon Sep 17 00:00:00 2001 From: Max Kostikov Date: Mon, 3 Dec 2018 21:33:32 +0100 Subject: Update Photo.php --- Zotlabs/Module/Photo.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'Zotlabs/Module') diff --git a/Zotlabs/Module/Photo.php b/Zotlabs/Module/Photo.php index a73492ae2..ddff3a68e 100644 --- a/Zotlabs/Module/Photo.php +++ b/Zotlabs/Module/Photo.php @@ -164,7 +164,7 @@ class Photo extends \Zotlabs\Web\Controller { ); if($cache_mode['on']) call_hooks('cache_url_hook', $cache); - if(! $cache['status'] && $cache_mode['leak']) { + if(! $cache['status']) { header("Location: " . htmlspecialchars_decode($r[0]['display_path'])); killme(); } -- cgit v1.2.3