Bug 372006 - [Patch] Support feed deltas
Summary: [Patch] Support feed deltas
Status: REPORTED
Alias: None
Product: frameworks-syndication
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: git
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-11-03 00:23 UTC by Daniel
Modified: 2018-07-01 07:53 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch (1.11 KB, patch)
2016-11-03 00:23 UTC, Daniel
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel 2016-11-03 00:23:09 UTC
Created attachment 101990 [details]
Patch

Support RFC3229 with the feed extension.

Instead of returning the full feed, supporting servers will return the delta since the If-Modified-Since timestamp or the If-None-Match ETag identifier. Reduces the bandwidth requirements for users and servers like, and reduces processing required as feeds only contain new entries since the last time they where checked.

The only modification required to support this in Akregator is adding the `A-IM: feed` header in requests when updating feeds. I’ve only tested this with Akregator, where it works as desired.

Question:
Does Syndication have other uses that I’m not aware of where the full feed would be preferred instead of deltas?
Comment 1 Laurent Montel 2016-11-03 07:03:06 UTC
Do you have patch for akregator too ?
Comment 2 Daniel 2016-11-03 09:49:41 UTC
> Do you have patch for akregator too?

No, this one patch in Syndication is all that is needed as Akregator already behaves as required. Well, the patch in bug 372005 is needed for this to work on non-English locales but that is a separate problem.
Comment 3 Laurent Montel 2016-11-04 06:54:57 UTC
So what is the improvment in akregator ?
What was the bug in akregator ?
Comment 4 Daniel 2016-11-04 08:55:33 UTC
Uhm, the improvement in Akregator is that supporting servers can return a feed delta with just new and modified entries instead of the full feed based on the request caching headers. Akregator already has a database of feed entries, and only cares about the new entries. The patch adds support for this by adding one extra header that signals support for this.
Comment 5 Daniel 2016-11-04 08:59:56 UTC
I see what the soruce of the confusion is now. When I say “The only modification required to support this in Akregator is adding the `A-IM: feed` header” I’m referring to this patch to Syndication that makes this available to Akregator.

Sorry for the poor choice of words.
Comment 6 Daniel 2018-02-10 03:51:32 UTC
Ping.