[PATCH] Add support for delta feeds

Development-related discussion, including bundled plugins
User avatar
noda
Bear Rating Trainee
Bear Rating Trainee
Posts: 2
Joined: 28 Oct 2016, 03:29

[PATCH] Add support for delta feeds

Postby noda » 28 Oct 2016, 03:43

Hi, his patch adds support for delta feeds (RFC3229+feeds). Supporting servers will return a delta feed instead of a full feed; meaning Tiny Tiny RSS will only be served entries that have been added or modified since the If-Modified-Since timestamp. Saves bandwidth for users and servers alike, and eases processing and speeds up feed updates for feed readers.

Supported in a a growing lists of feed readers and content management systems. I’m actively working on adding support to more clients and feed servers.

Patch:
delta-feeds.patch
(1.21 KiB) Downloaded 86 times


I’ve read the wiki page on contributing, and registered on GitLab with the username google (auto-assigned username when I logged in using my Google account). However, I don’t plan to contribute more patches and I’d appreciate it if someone could apply and merge the patch on my behalf.

User avatar
fox
^ me reading your posts ^
Posts: 6318
Joined: 27 Aug 2005, 22:53
Location: Saint-Petersburg, Russia
Contact:

Re: [PATCH] Add support for delta feeds

Postby fox » 28 Oct 2016, 08:18

tt-rss doesn't currently support if-modified-since for feeds so your patch is not really going to do anything at all, i'm afraid

since you didn't notice this while you were testing your submitted code this raises a few questions tbh

User avatar
noda
Bear Rating Trainee
Bear Rating Trainee
Posts: 2
Joined: 28 Oct 2016, 03:29

Re: [PATCH] Add support for delta feeds

Postby noda » 28 Oct 2016, 11:52

I thought it was due to some incompatibility with my environment (tested on FreeBSD), and didn’t look further in to it to be honest. I hard-coded the variable while testing and got the results I wanted. This patch will still add support when Tiny Tiny gets support for Last-Modified qualified caching. The new header just needs to tag along when the If-Modified-Since header is sent to work.


Return to “Development”

Who is online

Users browsing this forum: No registered users and 3 guests