[wp-trac] [WordPress Trac] #15936: IPv6 literal support in multisite broken

[WordPress Trac]

#15936: IPv6 literal support in multisite broken
---------------------------------------------+-----------------------------
 Reporter:  jmdh                             |       Owner:  SergeyBiryukov
     Type:  enhancement                      |      Status:  reviewing
 Priority:  normal                           |   Milestone:  5.4
Component:  Networks and Sites               |     Version:  3.0
 Severity:  major                            |  Resolution:
 Keywords:  has-patch has-unit-tests commit  |     Focuses:  multisite
---------------------------------------------+-----------------------------

Comment (by jeremyfelt):

 [attachment:"15936.4.diff"] is looking good.

 Using the docker-based development environment introduced in #47767, I
 added an mu-plugin that filtered allowed ports to support 8889. I was able
 to create an initial sub-directory network on `localhost:8889`, but new
 sites that I add are not immediately aware of their port number.

 The list of sites is now:

 1. `https://localhost:8889`
 2. `https://localhost/testing`

 I think this is acceptable in general as filters allow developers to
 adjust those URLs and I don't think there's an immediate need to start
 tracking port number in the database as part of a core solution to help
 with site lookup.

 We do probably need to figure out how to better support multisite in the
 new dev environment so that sites can be added. I'll hunt for a ticket or
 start one to track that.

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