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.