?
avatar tonypartridge
tonypartridge
3 Feb 2017

One for discussion really, the default htaccess being shipped by Joomla! since 1.6? change from that of 1.5. Now for every image that doesn't exist we are loading valuable php resources of the server. Shouldn't we be excluding images from with an exception.

Yes I know I can do this myself manually, but this is more for default installs where people do not know how to fine tune the htaccess.

example,

https://www.joomla.org/sdfsd.png

Loads a Joomla! 404. It shouldn't, it should be a server side 404 like we did on J1.5.

avatar tonypartridge tonypartridge - open - 3 Feb 2017
avatar joomla-cms-bot joomla-cms-bot - change - 3 Feb 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 3 Feb 2017
avatar mbabker
mbabker - comment - 3 Feb 2017

One of the features of Joomla and PHP is to serve more than HTML (sadly Joomla seems stuck on the HTML bit). You can serve images through PHP scripts (think dynamic trackers or ACL restricted stuff). So if a change is made, it should be based on file paths and not file extensions (so don't redirect requests for the images directory if a file doesn't exist versus not rewriting .png requests). Such a change also needs to be clearly documented or communicated to ensure developers using such a behavior aren't bitten in the backside).

avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Mar 2017
Category Installation
avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Apr 2017
Priority Medium Very low
Status New Needs Review
avatar brianteeman
brianteeman - comment - 21 May 2017

Are you going to submit a PR @tonypartridge or shall we treat the lack of comments as meaning no interest in making a change and it can be closed

avatar tonypartridge
tonypartridge - comment - 22 May 2017

I'm no

avatar tonypartridge tonypartridge - change - 22 May 2017
Status Needs Review Closed
Closed_Date 0000-00-00 00:00:00 2017-05-22 21:53:10
Closed_By tonypartridge
avatar tonypartridge tonypartridge - close - 22 May 2017

Add a Comment

Login with GitHub to post a comment