[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