[wp-trac] [WordPress Trac] #17376: Multisite Subfolders and bunk /wp-admin areas

[WordPress Trac]

#17376: Multisite Subfolders and bunk /wp-admin areas
-------------------------------------------------+-------------------------
 Reporter:  MadtownLems                          |       Owner:
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  4.2
Component:  Bootstrap/Load                       |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch dev-feedback needs-unit-   |     Focuses:  multisite
  tests                                          |
-------------------------------------------------+-------------------------
Changes (by jeremyfelt):

 * milestone:  Future Release => 4.2


Comment:

 Current status with valid sites at `http://foo.org/` and
 `http://foo.org/bar/`

 With [https://core.trac.wordpress.org/attachment/ticket/17376/17376.5.diff
 17376.5.diff]

 * `http://foo.org/wp-admin/` loads admin of main site.
 * `http://foo.org/bar/wp-admin/` loads admin of bar site.
 * `http://foo.org/typo/wp-admin/` redirects to `http://foo.org/`
 * `http://foo.org/bar/typo/wp-admin/` loads a 404 on bar site.

 With [https://core.trac.wordpress.org/attachment/ticket/17376/17376.4.diff
 17376.4.diff]

 * `http://foo.org/wp-admin/` loads admin of main site.
 * `http://foo.org/bar/wp-admin/` loads admin of bar site.
 * `http://foo.org/typo/wp-admin/` redirects to `http://foo.org/wp-admin/`
 * `http://foo.org/bar/typo/wp-admin/` loads a 404 on bar site.

 I like the simplicity of both patches in that we aren't trying to process
 a lot to figure out where the user is trying to go. I like the behavior of
 17376.5.diff the best as a mistyped `path/wp-admin/` loads a front end
 view. This forces another attempt rather than requiring the reading of an
 admin notice.

 We're probably ok with `http://foo.org/bar/typo/wp-admin/` loading a 404
 instead of redirecting in the case where `/bar/` is a valid path, not that
 WordPress is served from in `/bar/`.

 I'm betting we can find a way to write tests for this in
 `multisite/bootstrap.php`. :)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/17376#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform