Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: LimitInternalRecursion on Apache Multisite Subdomain Installs. #438

Closed
5 tasks done
alexbogias opened this issue Dec 5, 2022 · 7 comments
Closed
5 tasks done
Labels

Comments

@alexbogias
Copy link

Terms

Description

What's wrong?

I am having lots of apache errors from time to time in my Multisite Subdomain Installs.
AH00124: 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.

Steps To Reproduce

LimitInternalRecursion errors gives the following htaccess:

     RewriteEngine On
     RewriteRule .* - [E=HTTP_AUTHORIZATION:%{HTTP:Authorization}]
     RewriteBase /
     RewriteRule ^index\.php$ - [L]

     # add a trailing slash to /wp-admin
     RewriteRule ^wp-admin$ wp-admin/ [R=301,L]

     RewriteCond %{REQUEST_FILENAME} -f [OR]
     RewriteCond %{REQUEST_FILENAME} -d
     RewriteRule ^ - [L]
     RewriteRule ^(wp-.*.php)$ wp/$1 [L]
     RewriteRule ^(wp-(content|admin|includes).*) wp/$1 [L]
     RewriteRule ^(.*\.php)$ $1 [L]
     RewriteRule . index.php [L]

I am trying to work with RewriteBase /wp/ but I get file not found in /wp-login.php, wp-cron.php etc but no LimitInternalRecursion .

     RewriteEngine On
     RewriteRule .* - [E=HTTP_AUTHORIZATION:%{HTTP:Authorization}]
     RewriteBase /wp/
     RewriteRule ^index\.php$ - [L]

     # add a trailing slash to /wp-admin
     RewriteRule ^wp-admin$ wp-admin/ [R=301,L]

     RewriteCond %{REQUEST_FILENAME} -f [OR]
     RewriteCond %{REQUEST_FILENAME} -d
     RewriteRule ^ - [L]
     RewriteRule ^(wp-(content|admin|includes).*) $1 [L]
     RewriteRule ^(.*\.php)$ $1 [L]
     RewriteRule . index.php [L]

Expected Behavior

Not LimitInternalRecursion nor Primary script unknown on wp-*php

Actual Behavior

RewriteBase /wp/ gives Primary script unknown and RewriteBase / LimitInternalRecursion

Relevant Log Output

No response

Versions

Ubuntu 20.04 / Apache 2.4.41 / Wordpress 6.1.1 / Bedrcock 1.21.1

@alexbogias alexbogias added the bug label Dec 5, 2022
@swalkinshaw
Copy link
Member

This doesn't seem like a bug in Bedrock since we have no included Apache integration or htaccess file. Why do you think it's a bug in Bedrock itself?

@alexbogias
Copy link
Author

alexbogias commented Dec 5, 2022

@swalkinshaw What do you mean? /wp in urls is bedrock thing i suppose. Doesnt need any apache rules?
https://docs.roots.io/bedrock/master/server-configuration/#apache-configuration-for-bedrock
Here you have nginx regular wp multisite, subpath and multisite subdomain configuration. Apache misses both multisite required configurations.

@swalkinshaw
Copy link
Member

Our docs are in https://github.com/roots/docs if you think this is an issue with the documentation? I'm not sure if you're saying that example config is wrong or just missing an improvement.

But there's nothing to fix in this repo for Bedrock itself.

@alexbogias
Copy link
Author

alexbogias commented Dec 5, 2022

Docs are missing apache multisite configuration, and I am using unofficial third-party .htaccess snippets form the forum, but nothing seems to work without issues...

@alexbogias
Copy link
Author

alexbogias commented Dec 5, 2022

the example config is for non-multisite installs and is working fine, Why closed?

@swalkinshaw swalkinshaw transferred this issue from roots/bedrock Dec 5, 2022
@swalkinshaw swalkinshaw reopened this Dec 5, 2022
@swalkinshaw
Copy link
Member

I transferred the issue to https://github.com/roots/docs

@retlehs
Copy link
Member

retlehs commented Dec 5, 2022

Doing the research and maintaining the proper Apache webserver configuration for WordPress multisite setups isn't something that we're interested in. None of us use Apache.

Continue this discussion on the 6 year old topic that you bumped prior to submitting this issue.

@retlehs retlehs closed this as not planned Won't fix, can't repro, duplicate, stale Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants