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

Powered by eZ Publish™ CMS Open Source Web Content Management. Copyright © 1999-2014 eZ Systems AS (except where otherwise noted). All rights reserved.

eZ debug

Timing: Jan 18 2025 00:45:52
Script start
Timing: Jan 18 2025 00:45:52
Module start 'layout'
Timing: Jan 18 2025 00:45:52
Module start 'content'
Timing: Jan 18 2025 00:45:53
Module end 'content'
Timing: Jan 18 2025 00:45:53
Script end

Main resources:

Total runtime1.1625 sec
Peak memory usage4,096.0000 KB
Database Queries66

Timing points:

CheckpointStart (sec)Duration (sec)Memory at start (KB)Memory used (KB)
Script start 0.00000.0048 589.2891152.6563
Module start 'layout' 0.00490.0024 741.945339.5156
Module start 'content' 0.00731.1539 781.4609627.1328
Module end 'content' 1.16120.0012 1,408.593816.0938
Script end 1.1624  1,424.6875 

Time accumulators:

 Accumulator Duration (sec) Duration (%) Count Average (sec)
Ini load
Load cache0.00310.2660160.0002
Check MTime0.00120.1035160.0001
Mysql Total
Database connection0.00060.053310.0006
Mysqli_queries1.098594.4932660.0166
Looping result0.00070.0608640.0000
Template Total1.132397.420.5661
Template load0.00220.190120.0011
Template processing1.130197.211820.5650
Template load and register function0.00010.009910.0001
states
state_id_array0.00100.089510.0010
state_identifier_array0.00230.198420.0012
Override
Cache load0.00190.1659440.0000
Sytem overhead
Fetch class attribute can translate value0.00050.043830.0002
Fetch class attribute name0.00120.099180.0001
XML
Image XML parsing0.00160.136630.0005
class_abstraction
Instantiating content class attribute0.00000.0014100.0000
General
dbfile0.00100.0901310.0000
String conversion0.00000.000740.0000
Note: percentages do not add up to 100% because some accumulators overlap

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
1print_pagelayout.tpl<No override>extension/community/design/community/templates/print_pagelayout.tplEdit templateOverride template
 Number of times templates used: 29
 Number of unique templates used: 8

Time used to render debug report: 0.0001 secs