[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