[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 05:49:47 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 |
--------------------------------+-----------------------
Changes (by rmccue):
* owner: => rmccue
* status: new => assigned
Comment:
Replying to [comment:7 nacin]:
> For 1.2.0, I get:
> {{{
> string(143)
"https://news.google.com/news/feeds?gl%3Dvn%26pz%3D1%26cf%3Dall%26ned%3Dvi_vn%26hl%3Dvi%26q%3Dch%E1%BB%91ng+h%C3%A0ng+gi%E1%BA%A3%26output%3Drss"
> }}}
>
> Not sure if that is right or not. Seems like it worked in 3.3, so I
imagine yes.
Nope, that's incorrect. Note how the & and = in the query string have been
encoded. That was issue 112, which was fixed. This appears to be a new
issue, but related.
This appears to be related to
https://github.com/simplepie/simplepie/issues/162 and possibly
https://github.com/simplepie/simplepie/issues/58
From the issue description:
{{{
http://news.google.com/news?gl=vn&pz=1&cf=all&ned=vi_vn&hl=vi&q=ch%E1%BB%91ng+h%C3%A0ng+gi%E1%BA%A3&output=rss
http://news.google.com/news?gl=vn&pz=1&ned=vi_vn&hl=vi&q=ch%EF%BF%BD%EF%BF%BD%EF%BF%BDng+h%EF%BF%BD%EF%BF%BDng+gi%EF%BF%BD%EF%BF%BD%EF%BF%BD
}}}
Note that `&cf=all` and `&output=rss` have disappeared. All the characters
in the original URL have been turned into a � character (U+FFFD, aka 0xEF
0xBF 0xBD). That would indicate to me a possible issue with iconv or
mbstring on the version of PHP on the server. I'll have a dig into this
when I've got stable internet.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/20999#comment:8>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list