[wp-trac] [WordPress Trac] #37763: Target server overload due to invalid RSS feed URL in RSS widget

[WordPress Trac]

#37763: Target server overload due to invalid RSS feed URL in RSS widget
--------------------------+------------------------------
 Reporter:  bstovall      |       Owner:  stevenkword
     Type:  defect (bug)  |      Status:  assigned
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Feeds         |     Version:  4.6
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |     Focuses:  performance
--------------------------+------------------------------

Comment (by stevenkword):

 Replying to [comment:10 boonebgorges]:
 > SimplePie has internal caching for successfully parsed RSS feeds.
 Perhaps in the WP RSS widget - or even lower in the stack, perhaps in
 `fetch_feed()` - we can add our own caching (say, 10 minutes) for
 "invalid" feeds (where "invalid" covers both 404s and documents that can't
 be parsed as RSS). While not perfect, this would minimize the potential
 for self-DoSing. What do others think?

 This seems like a good iterative approach.  @peterchester has a good start
 on it over in #11585, but it needs a refresh.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/37763#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform