[wp-trac] [WordPress Trac] #16859: esc_url eats square brackets.

WordPress Trac noreply at wordpress.org
Thu Feb 20 23:44:47 UTC 2014


#16859: esc_url eats square brackets.
------------------------------------+-----------------------------
 Reporter:  f00f                    |       Owner:  westi
     Type:  defect (bug)            |      Status:  accepted
 Priority:  normal                  |   Milestone:  Future Release
Component:  Formatting              |     Version:  3.1
 Severity:  major                   |  Resolution:
 Keywords:  has-patch dev-feedback  |     Focuses:
------------------------------------+-----------------------------

Comment (by nacin):

 This looks like it could be quite a bit slower. Any data on that?

 One thing we've thought about is having a "common situation" regex that
 only allows very basic characters and declares the URL as safe if it
 matches. Then do the more rigorous, slower checks if it doesn't. It'll
 slightly slow down the more complex URLs but will speed up the rest of
 them. The ticket for this is #22951. If the work here significantly slows
 down this function, it should probably be considered hand-in-hand with
 #22951.

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


More information about the wp-trac mailing list