You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Please review the original issue and especially its comments. Comments here on closed issues will be ignored. Thanks.
Original description
The generated htaccess file contains a couple of inefficiencies.
1) Rewrites are enclosed in a <IfModule> block. Since the code doesn't work without mod_rewrite, we *want* it to fail if mod_rewrite is absent, not silently ignore our configuration and cause confusion. Thus, patch removes the <IfModule> stuff.
2) The RewriteRule has a (.*) in it, which is unnecessary. (.*) captures a match in a backreference for later use. We're not using that backreference, so this just adds memory and time overhead on every request. Granted, it's not much, but there it is.
(Note: Yes, these changes are trivial, but mod_rewrite inefficiency bugs me.)
(Note 2: The entire .htaccess file can be replaced with a single "FallBackResource $path/yourls-loader.php" if you're running Apache 2.2.16 or later.)
The text was updated successfully, but these errors were encountered:
This is a COPY of Issue 1300: PATCH - Small changes to the generated .htaccess file, filed on Google Code before the project was moved on Github.
Please review the original issue and especially its comments. Comments here on closed issues will be ignored. Thanks.
Original description
The text was updated successfully, but these errors were encountered: