[wp-trac] [WordPress Trac] #20999: WordPress 3.4 doesn't handle rss url with percent code correctly

WordPress Trac wp-trac at lists.automattic.com
Tue Jun 26 07:12:54 UTC 2012


#20999: WordPress 3.4 doesn't handle rss url with percent code correctly
--------------------------------+-----------------------
 Reporter:  herophuong          |       Owner:  rmccue
     Type:  defect (bug)        |      Status:  assigned
 Priority:  normal              |   Milestone:  3.4.1
Component:  External Libraries  |     Version:  3.4
 Severity:  normal              |  Resolution:
 Keywords:  dev-feedback        |
--------------------------------+-----------------------

Comment (by rmccue):

 The commit which broke this was
 [https://github.com/simplepie/simplepie/commit/4e022de1271dda53c3c5bcec8537554b0eb5240b
 4e022de12] for 1.3-dev and
 [https://github.com/simplepie/simplepie/commit/76b5fd632f40c4516d68f3f1bdabcd76829117cc
 76b5fd63] for 1.2.1. This commit was a backport of the more up-to-date
 IRI-parsing code from the ComplexPie fork (possibly future SimplePie 2).

 The issue is that SimplePie thinks the characters are valid for a URL
 [https://github.com/simplepie/simplepie/commit/76b5fd632f40c4516d68f3f1bdabcd76829117cc#L0R12013
 when it checks]. While they are valid UTF-8/UCS characters, they don't
 appear to be valid for URLs based on my reading of
 [http://tools.ietf.org/html/rfc3986 RFC3986]. At the moment, I'm lacking
 the mental power to work out exactly why that test is there, so I'll have
 to come back to doing this later.

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


More information about the wp-trac mailing list