Hello.
I have some issues with current image cache realization. Could you make it more powerful please?
1. I can't set up or disable cache expiration time without hadrcode it. Could you make a config variable for it? (Now it's like this: time() - filemtime($file) > 86400*7)
2. I can't save image with human readable filename. You fixed it in plugin, could you port it to general code? https://tt-rss.org/gitlab/fox/tt-rss/co ... 05214fa568
3. Modern irc-clients and stuff like Discord can't make a preview of image cause it doesn't look like an image url. Could you append mime-based extension to filename (like /bla/bla/ad5647fda...567adcab.png)?
3.1. Content-type is hardcoded to image/png, could you show a real mime of a file FTGJ?
3.2. Optimisation for disabled cache expiration. Could you add some subdirectories like some CMS and nginx do for cache (like '/images/' . substr($hash, 1) . substr($hash, 1, 2) . $hash)
4. Direct link to files. I'd like to show files with nginx only without touching php at all. Could you change link format for more nginx-friendly?
Thank you.
Cache improvements
- fox
- ^ me reading your posts ^
- Posts: 6318
- Joined: 27 Aug 2005, 22:53
- Location: Saint-Petersburg, Russia
- Contact:
Re: Cache improvements
personally i think image caching is a minor feature which works fine as it is. it was never intended for someone to just download everything on the internet and keep it forever, which would be stupid and pointless. thus, current design.
now, detailed comments:
1. i don't think adding a tunable here is a good idea, config.php is bloated enough as it is. especially, because someone will definitely set it to some retarded value and then come complaining tt-rss ate all his disk space or w/e.
2, 3. this sounds reasonable and easy to add
3.1. there's nothing to be gained here because any browser is going to figure the real image by contents, and downloaders wouldn't care
3.2. no, this makes things harder to clean up for no good reason. again i'm not aiming for some kind of industrial kitty meme download factory here, you have to understand the intended scope of this feature.
4. no, tt-rss in general is not giving direct access to files in cache directories.
now, detailed comments:
1. i don't think adding a tunable here is a good idea, config.php is bloated enough as it is. especially, because someone will definitely set it to some retarded value and then come complaining tt-rss ate all his disk space or w/e.
2, 3. this sounds reasonable and easy to add
3.1. there's nothing to be gained here because any browser is going to figure the real image by contents, and downloaders wouldn't care
3.2. no, this makes things harder to clean up for no good reason. again i'm not aiming for some kind of industrial kitty meme download factory here, you have to understand the intended scope of this feature.
4. no, tt-rss in general is not giving direct access to files in cache directories.
Re: Cache improvements
Works fine. Thank you.
- fox
- ^ me reading your posts ^
- Posts: 6318
- Joined: 27 Aug 2005, 22:53
- Location: Saint-Petersburg, Russia
- Contact:
Re: Cache improvements
this is going to change somewhat. i'm changing cache handler to set content-disposition: inline because it makes it possible to watch cached urls directly in browser without mandatory auto-saving.
the filename is still going to be there if user presses ctrl-s etc. i think it's a less annoying behavior overall for media.
the filename is still going to be there if user presses ctrl-s etc. i think it's a less annoying behavior overall for media.
Who is online
Users browsing this forum: No registered users and 1 guest