[wp-trac] [WordPress Trac] #23605: esc_url() strips spaces instead of encoding them

WordPress Trac noreply at wordpress.org
Thu Jan 23 23:55:21 UTC 2014


#23605: esc_url() strips spaces instead of encoding them
--------------------------+-----------------------------
 Reporter:  johnbillion   |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Future Release
Component:  Formatting    |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+-----------------------------

Comment (by betzster):

 Replying to [comment:11 nacin]:
 > If we're going to encode spaces, should we be encoding other characters
 too? I'm not necessarily against this fix — if a space is passed to
 esc_url(), we know exactly what it should be and it shouldn't be a big
 deal to encode it — but I'm wondering where that stops.

 Can't we just lean on `rawurlencode()` to encode everything properly?

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


More information about the wp-trac mailing list