Adding example RewriteBase to static/.htaccess for Apache users

[steverobbins]

Similar to the example configuration in pub/.htaccess, I'd like to add one here. My local install wouldn't work without it. This hint would have helped me track down the issue sooner.

[cmuench]

+1

[antonkril]

What's the issue with local install? Why would it not work without it. Can you describe problem more?

[vancoz]

@steverobbins could you please address @antonkril 's question?

[steverobbins]

@antonkril Without it, I get the following error in /var/log/httpd/error_log

[Tue Sep 15 17:34:12 2015] [error] [client 192.168.50.1] Request exceeded the limit of 10 internal redirects due to probable configuration error. Use 'LimitInternalRecursion' to increase the limit if necessary. Use 'LogLevel debug' to get a backtrace.

I notice this behavior when using MAMP or CentOS 6.5 with Apache 2.2.15 (vagrant)

[buskamuza]

@steverobbins , could you, please, provide your Apache configuration, including configuration for the virtual host?
As I understand, you have Magento installed under <document_root>/magento? Do you use something like http://httpd.apache.org/docs/2.2/vhosts/mass.html or other configuration for multiple virtual hosts?

So we could better document in which case this setting is needed.

[steverobbins]

I will work on getting you the files you asked for, but really I don't feel I need to explain this any more than I need to explain why it's already in these places. I'm just trying to keep things consistent.

[steverobbins]

Here are a few articles on the same issue

• http://www.andrew-kirkpatrick.com/2012/06/apache-virtualhost-htaccess-issue-request-exceeded-the-limit-of-10-internal-redirects-redirected-from-r-uri/
• https://processwire.com/talk/topic/5361-apache-recursion-problem/
• http://willcodeforcoffee.com/2007/01/31/cakephp-error-500-too-many-redirects/

The solution for these users was to add a RewriteBase or to change

RewriteRule .* index.php [L]

to

RewriteRule .* /index.php [L]

Here are relevant configuration files: https://gist.github.com/steverobbins/d799bffe47719083db7e

Again, I just want to add this commented out line so that users who experience this issue will have a hint as to how to solve it. This is consistent with the main .htaccess file. It will not effect any existing installs.

[magento-cicd2]

We have automated a Magento Contributor License Agreement verifier for contributions sent to our GitHub projects.
Please see the CLA agreement in the Pull Request comments below.

[steverobbins]

This has been marked as accept for over two months. Will it be merged in?

[KrystynaKabannyk]

hello @steverobbins, thanks so much for your pull request, the proposed changes look good to us, and we plan to merge it in one of the nearest releases. However, we would highly appreciate if you update the code and cover it by automated tests (the checks circle has to be green), so we could merge it earlier.

[steverobbins]

@KrystynaKabannyk If you look at the contents of this PR you will see that

• The changes don't warrant a test
• The changes aren't testable
• The tests are failing for issues unrelated to this PR

Regardless, I have rebased on develop to trigger a new test.

[steverobbins]

@VladimirZaets Is there a reason why this was rejected? It's been marked as accept for some time.