[buddypress-trac] [BuddyPress] #5213: Set better HTTP headers for RSS feeds

buddypress-trac noreply at wordpress.org
Thu Oct 24 20:51:39 UTC 2013

#5213: Set better HTTP headers for RSS feeds
 Reporter:  r-a-y         |       Owner:  r-a-y
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  1.9
Component:  Activity      |     Version:  1.8
 Severity:  normal        |  Resolution:  fixed
 Keywords:                |
Changes (by r-a-y):

 * owner:   => r-a-y
 * status:  new => closed
 * resolution:   => fixed


 In [changeset:"7477"]:
 #!CommitTicketReference repository="" revision="7477"
 Set better HTTP headers for activity RSS feeds.

 Previously, BP's RSS feeds would use WP's native HTTP headers set in the
 WP::send_headers() method.  This would cause BuddyPress to inherit WP
 feed's "ETag" and HTTP status headers.

 When trying to refresh BP's RSS feeds, this would not work correctly due
 to BP relying on WP's post last modified date instead of BP's activity
 updated date.  This would lead to a HTTP 304 Not Modified status header
 being sent by WP, meaning the client (browser, RSS reader) would not fetch
 the latest items from the activity stream.

 This commit sets the appropriate HTTP headers for BP's activity RSS feeds
 and uses a new, internal class method - http_headers() - to set them.

 Fixes #5213.

Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5213#comment:1>
BuddyPress <http://buddypress.org/>

More information about the buddypress-trac mailing list