[wp-trac] [WordPress Trac] #12136: Improve MS rewrite rules

WordPress Trac wp-trac at lists.automattic.com
Thu Mar 11 04:44:11 UTC 2010


#12136: Improve MS rewrite rules
---------------------------------+------------------------------------------
 Reporter:  nacin                |        Owner:  wpmuguru
     Type:  defect (bug)         |       Status:  reopened
 Priority:  normal               |    Milestone:  3.0     
Component:  Rewrite Rules        |      Version:          
 Severity:  normal               |   Resolution:          
 Keywords:  multisite has-patch  |  
---------------------------------+------------------------------------------
Changes (by nacin):

  * status:  closed => reopened
  * resolution:  wontfix =>


Comment:

 Going to re-open this for some incremental improvements. The bogus path
 works on the current rewrite rules as well.

 There are numerous improvements that seem okay to me. In order:

 1. Remove {{{RewriteRule ^(.*/)?files/$ index.php [L]}}} and change
 {{{files/(.*)}}} in the second rule to {{{files/(.+)}}}.

 2. Remove the condition on the wp-admin trailing slash rule by improving
 the rule itself.

 3. Convert the files and wp-admin rules from {{{^(.*/)?}}} to {{{^([_0-9a-
 zA-Z-]+/)?}}}, to match the other rules and current restrictions.

 4. Remove the wp-content/plugins restriction on files, as the conditions
 for which it was added shouldn't apply under a properly constructed rule.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/12136#comment:8>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list