Ticket #3513 (closed enhancement: fixed)

Opened 5 years ago

Last modified 3 years ago

RSS feeds need updateFrequency to limit scans by Outlook

Reported by: _ck_ Owned by: westi
Priority: low Milestone: 2.7
Component: Template Version: 2.0
Severity: trivial Keywords: has-patch dev-feedback needs-testing
Cc:

Description

I discovered a few visitors were pulling down the RSS feeds thousands of times each day without realizing it. This turned out to be because they are using the new Office 12/2007 update with IE7 and that makes it pull down the entire feed every time they check mail, sometimes every 10 minutes or less (thanks Microsoft!)

This problem is about to become widespread when many more people upgrade.

RSS feed needs this namespace added

xmlns:sy="http://purl.org/rss/1.0/modules/syndication/"

then specify an hourly frequency maximum check time:

<sy:updatePeriod> hourly </sy:updatePeriod>
<sy:updateFrequency> 1 </sy:updateFrequency>

Comments Feed needs the addition as well, perhaps atom feeds too.

Attachments

3513.diff Download (2.1 KB) - added by rob1n 5 years ago.

Change History

  • Milestone changed from 2.0.6 to 2.2
  • Milestone changed from 2.2 to 2.3
  • Keywords has-patch added

+1

This would really help reduce the load due to some stupid feed readers without resorting to feedburner, which is what a lot of people have done to deal with it.

  • Owner changed from anonymous to rob1n
  • Status changed from new to assigned

rob1n5 years ago

  • Keywords commit added
  • Keywords commit removed

I guess the patch solves it for the RSS 2.0 feeds (posts and comments).

  • Keywords dev-feedback added
  • Owner rob1n deleted
  • Status changed from assigned to new
  • Keywords needs-testing added
  • Milestone changed from 2.5 to 2.6
  • Milestone changed from 2.9 to 2.6.1

I seriously cannot believe it's taken TWO YEARS to include this trivial fix for a fairly common issue given the sheer number of Outlook users. Is it just being missed?

  • Priority changed from normal to low
  • Severity changed from normal to trivial
  • Milestone changed from 2.6.1 to 2.7

Gots to show tickets some love for them to get in. Does seem pretty easy. Moving to trunk.

  • Owner set to westi
  • Status changed from new to assigned

The patch looks good to me for trunk.

I think we may want to do something cleverer than setting it to every hour though.

Maybe we can easily pre-compute something on the post publish hook which says how often a site is updated e.g. posts per month / hours in month

Or maybe we can expose a single ui option?

I'll get the patch in but leave this open for further discussion.

(In [8619]) Add syndication frequency recommendations to rss2 feeds. See #3513 props _ck_.

Is this fixed? The patch seems to be committed.

  • Status changed from assigned to closed
  • Resolution set to fixed

Yep.

Note: See TracTickets for help on using tickets.