Forums / Developer / default rewrite rules not working properly with content/download

default rewrite rules not working properly with content/download

Author Message

Marko Žmak

Wednesday 13 October 2010 10:44:08 am

The suggested rewrite rule that comes with default eZP installation in .htaccess_root:

RewriteRule !(\.(gif|jpe?g?|png|css|js|swf|html?)|var(.+)storage.pdf(.+)\.pdf)$ index.php

is not working properly. The problem is when you upload for example a .jpg image as an object of File class. The uploaded image can then be uploaded via an URL that goes like this:

/content/download/7151474/29980970/version/1/file/mypic.jpg

So because the URL ends with ".jpg", the it is not rewritten with index.php, and you get a 404 page.

As a quickfix I made this change:

RewriteRule !(design|var|share)/.*\.(gif|jpe?g|png|css|js|html)|var(.+)storage.pdf(.+)\.pdf$ index.php 

but this won't work if you have an eZ node with a path like this:

/var/some_node.jpg

Another idea is to have this rewrite rules:

RewriteRule ^.*content/download/.*$ index.php
RewriteRule !*\.(gif|jpe?g|png|css|js|html)|var(.+)storage.pdf(.+)\.pdf$ index.php 

But this will not work for static images URLs like this:

/design/content/download/some_image.jpg

Also a change to this last rule:

RewriteRule ^content/download/.*$ index.php
RewriteRule !*\.(gif|jpe?g|png|css|js|html)|var(.+)storage.pdf(.+)\.pdf$ index.php 

that matches only URLs that begin with "content/download" won't work because if you have multiple siteacces, it won't rewrite the URLs that begin with the siteaccess name.

I'll think about this further, but...

Anyone already has a better idea?

Can we get a rewrite rule that will work for all the cases?

--
Nothing is impossible. Not if you can imagine it!

Hubert Farnsworth

Gaetano Giunta

Wednesday 13 October 2010 10:54:21 am

Simple: only use location-based rules instead of extension-based ones.

1. add location-based rules for all extensions design dirs, for those cache dirs that are to be accessed (but NOT to var/cache/* mind you), etc

2. the last catch-all rule does not even need to analyze file extension

Principal Consultant International Business
Member of the Community Project Board

Gaetano Giunta

Wednesday 13 October 2010 10:55:17 am

ps: you might find a working example in the rewrite rules commonly used for vhost setups (somewhere in the docs or on this site)

Principal Consultant International Business
Member of the Community Project Board

André R.

Thursday 14 October 2010 1:01:34 am

Try this one:

http://doc.ez.no/eZ-Publish/Technical-manual/3.10/Installation/Virtual-host-setup#comment15233

eZ Online Editor 5: http://projects.ez.no/ezoe || eZJSCore (Ajax): http://projects.ez.no/ezjscore || eZ Publish EE http://ez.no/eZPublish/eZ-Publish-Enterprise-Subscription
@: http://twitter.com/andrerom

Marko Žmak

Sunday 17 October 2010 11:48:49 am

"

Simple: only use location-based rules instead of extension-based ones.

1. add location-based rules for all extensions design dirs, for those cache dirs that are to be accessed (but NOT to var/cache/* mind you), etc

2. the last catch-all rule does not even need to analyze file extension

"

Hm, why is then this rule made like this in the default .htaccess_root that comes with eZP?

Why don't we have by default the correct rewrite rules in .htaccess_root?

--
Nothing is impossible. Not if you can imagine it!

Hubert Farnsworth

eZ debug

Timing: Jan 17 2025 22:41:11
Script start
Timing: Jan 17 2025 22:41:11
Module start 'content'
Timing: Jan 17 2025 22:41:12
Module end 'content'
Timing: Jan 17 2025 22:41:12
Script end

Main resources:

Total runtime0.9245 sec
Peak memory usage4,096.0000 KB
Database Queries203

Timing points:

CheckpointStart (sec)Duration (sec)Memory at start (KB)Memory used (KB)
Script start 0.00000.0075 589.3984180.8047
Module start 'content' 0.00750.7842 770.2031632.2422
Module end 'content' 0.79170.1328 1,402.4453341.3359
Script end 0.9244  1,743.7813 

Time accumulators:

 Accumulator Duration (sec) Duration (%) Count Average (sec)
Ini load
Load cache0.00430.4607210.0002
Check MTime0.00150.1655210.0001
Mysql Total
Database connection0.00140.146810.0014
Mysqli_queries0.838590.69182030.0041
Looping result0.00230.25402010.0000
Template Total0.893396.620.4466
Template load0.00210.223020.0010
Template processing0.891296.395920.4456
Template load and register function0.00010.009310.0001
states
state_id_array0.00170.181110.0017
state_identifier_array0.00090.098720.0005
Override
Cache load0.00180.1974440.0000
Sytem overhead
Fetch class attribute can translate value0.00150.162840.0004
Fetch class attribute name0.00120.134480.0002
XML
Image XML parsing0.00190.206140.0005
class_abstraction
Instantiating content class attribute0.00000.0025100.0000
General
dbfile0.00330.3608420.0001
String conversion0.00000.000630.0000
Note: percentages do not add up to 100% because some accumulators overlap

CSS/JS files loaded with "ezjscPacker" during request:

CacheTypePacklevelSourceFiles
CSS0extension/community/design/community/stylesheets/ext/jquery.autocomplete.css
extension/community_design/design/suncana/stylesheets/scrollbars.css
extension/community_design/design/suncana/stylesheets/tabs.css
extension/community_design/design/suncana/stylesheets/roadmap.css
extension/community_design/design/suncana/stylesheets/content.css
extension/community_design/design/suncana/stylesheets/star-rating.css
extension/community_design/design/suncana/stylesheets/syntax_and_custom_tags.css
extension/community_design/design/suncana/stylesheets/buttons.css
extension/community_design/design/suncana/stylesheets/tweetbox.css
extension/community_design/design/suncana/stylesheets/jquery.fancybox-1.3.4.css
extension/bcsmoothgallery/design/standard/stylesheets/magnific-popup.css
extension/sevenx/design/simple/stylesheets/star_rating.css
extension/sevenx/design/simple/stylesheets/libs/fontawesome/css/all.min.css
extension/sevenx/design/simple/stylesheets/main.v02.css
extension/sevenx/design/simple/stylesheets/main.v02.res.css
JS0extension/ezjscore/design/standard/lib/yui/3.17.2/build/yui/yui-min.js
extension/ezjscore/design/standard/javascript/jquery-3.7.0.min.js
extension/community_design/design/suncana/javascript/jquery.ui.core.min.js
extension/community_design/design/suncana/javascript/jquery.ui.widget.min.js
extension/community_design/design/suncana/javascript/jquery.easing.1.3.js
extension/community_design/design/suncana/javascript/jquery.ui.tabs.js
extension/community_design/design/suncana/javascript/jquery.hoverIntent.min.js
extension/community_design/design/suncana/javascript/jquery.popmenu.js
extension/community_design/design/suncana/javascript/jScrollPane.js
extension/community_design/design/suncana/javascript/jquery.mousewheel.js
extension/community_design/design/suncana/javascript/jquery.cycle.all.js
extension/sevenx/design/simple/javascript/jquery.scrollTo.js
extension/community_design/design/suncana/javascript/jquery.cookie.js
extension/community_design/design/suncana/javascript/ezstarrating_jquery.js
extension/community_design/design/suncana/javascript/jquery.initboxes.js
extension/community_design/design/suncana/javascript/app.js
extension/community_design/design/suncana/javascript/twitterwidget.js
extension/community_design/design/suncana/javascript/community.js
extension/community_design/design/suncana/javascript/roadmap.js
extension/community_design/design/suncana/javascript/ez.js
extension/community_design/design/suncana/javascript/ezshareevents.js
extension/sevenx/design/simple/javascript/main.js

Templates used to render the page:

UsageRequested templateTemplateTemplate loadedEditOverride
1node/view/full.tplfull/forum_topic.tplextension/sevenx/design/simple/override/templates/full/forum_topic.tplEdit templateOverride template
5content/datatype/view/ezimage.tpl<No override>extension/sevenx/design/simple/templates/content/datatype/view/ezimage.tplEdit templateOverride template
5content/datatype/view/ezxmltext.tpl<No override>extension/community_design/design/suncana/templates/content/datatype/view/ezxmltext.tplEdit templateOverride template
11content/datatype/view/ezxmltags/paragraph.tpl<No override>extension/ezwebin/design/ezwebin/templates/content/datatype/view/ezxmltags/paragraph.tplEdit templateOverride template
4content/datatype/view/ezxmltags/literal.tpl<No override>extension/community/design/standard/templates/content/datatype/view/ezxmltags/literal.tplEdit templateOverride template
1content/datatype/view/ezxmltags/link.tpl<No override>design/standard/templates/content/datatype/view/ezxmltags/link.tplEdit templateOverride template
1content/datatype/view/ezxmltags/quote.tpldatatype/ezxmltext/quote.tplextension/ezwebin/design/ezwebin/override/templates/datatype/ezxmltext/quote.tplEdit templateOverride template
1pagelayout.tpl<No override>extension/sevenx/design/simple/templates/pagelayout.tplEdit templateOverride template
 Number of times templates used: 29
 Number of unique templates used: 8

Time used to render debug report: 0.0002 secs