Ori.livneh has submitted this change and it was merged.
Change subject: UrlShortener on beta: fix RewriteRule
......................................................................
UrlShortener on beta: fix RewriteRule
'/' matches both the 'Redirect /' directive and 'RewriteRule ^/(.*)' directive,
because '*' is '0 or more'. Changing it to '+' (one or more) resolves the
ambiguity and makes Apache not freak out.
Change-Id: Icb1163878412fcaeb969c9166c160df051104783
---
M modules/mediawiki/files/apache/beta/sites/main.conf
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Ori.livneh: Verified; Looks good to me, approved
Legoktm: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/modules/mediawiki/files/apache/beta/sites/main.conf
b/modules/mediawiki/files/apache/beta/sites/main.conf
index 8d375b3..e10e8b8 100644
--- a/modules/mediawiki/files/apache/beta/sites/main.conf
+++ b/modules/mediawiki/files/apache/beta/sites/main.conf
@@ -24,5 +24,5 @@
Redirect / http://meta.wikimedia.beta.wmflabs.org/wiki/Special:UrlShortener
RewriteEngine On
- RewriteRule ^/(.*)$
meta.wikimedia.beta.wmflabs.org/w/index.php?title=Special:UrlRedirector/$1 [PT]
+ RewriteRule ^/(.+)$
meta.wikimedia.beta.wmflabs.org/w/index.php?title=Special:UrlRedirector/$1 [PT]
</VirtualHost>
--
To view, visit https://gerrit.wikimedia.org/r/248607
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icb1163878412fcaeb969c9166c160df051104783
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ori.livneh <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits