[wp-trac] [WordPress Trac] #11219: WordPress should cache failed feed fetches so as to avoid overloading feed sources (was: SimplePie fails when trying to convert UTF-8 to UTF-8)

[WordPress Trac]

#11219: WordPress should cache failed feed fetches so as to avoid overloading feed
sources
-------------------------------+--------------------------------------------
 Reporter:  Denis-de-Bernardy  |       Owner:             
     Type:  defect (bug)       |      Status:  new        
 Priority:  normal             |   Milestone:  3.0        
Component:  Optimization       |     Version:  2.9        
 Severity:  normal             |    Keywords:  needs-patch
-------------------------------+--------------------------------------------
Changes (by westi):

  * keywords:  => needs-patch
  * milestone:  2.9 => 3.0


Old description:

> attached patch fixes the issue highlighted in the screenshot.

New description:

 In some situations SimplePie cannot parse a feed due to it being malformed
 we should cache that feed so as to not DOS servers.

--

Comment:

 The current patch doesn't seem to be the right thing to do here.

 SimplePie has determined that it wants to change the encoding and if this
 fails the function is meant to return false.

 As for the caching of failed lookups it is something we could do but not
 for 2.9.

 We need to work out what we should cache, and for how long.

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