{"action":"closed","number":44,"pull_request":{"url":"https://api.github.com/repos/plone/plone.app.portlets/pulls/44","id":23355146,"html_url":"https://github.com/plone/plone.app.portlets/pull/44","diff_url":"https://github.com/plone/plone.app.portlets/pull/44.diff","patch_url":"https://github.com/plone/plone.app.portlets/pull/44.patch","issue_url":"https://api.github.com/repos/plone/plone.app.portlets/issues/44","number":44,"state":"closed","locked":false,"title":"Use caching headers while getting rss, remove unconfigurable additional delay on errors","user":{"login":"do3cc","id":147512,"avatar_url":"https://avatars.githubusercontent.com/u/147512?v=3","gravatar_id":"","url":"https://api.github.com/users/do3cc","html_url":"https://github.com/do3cc","followers_url":"https://api.github.com/users/do3cc/followers","following_url":"https://api.github.com/users/do3cc/following{/other_user}","gists_url":"https://api.github.com/users/do3cc/gists{/gist_id}","starred_url":"https://api.github.com/users/do3cc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/do3cc/subscriptions","organizations_url":"https://api.github.com/users/do3cc/orgs","repos_url":"https://api.github.com/users/do3cc/repos","events_url":"https://api.github.com/users/do3cc/events{/privacy}","received_events_url":"https://api.github.com/users/do3cc/received_events","type":"User","site_admin":false},"body":"Unfortunately, I did not break up the two changes in two different PRs, but being quite small, I hope it is not necessary.\r\nChange 1: Honour caching headers when getting rss requests.\r\nChange 2: Remove the hard coded additional ten minute delay for polling rss feeds again.\r\n\r\nChange 2 hard coded additional delays on error is a PITA if you use rss to tightly couple two internal systems and want a low delay of 1 minute only.\r\nOtoh if you use it to get rss feeds from some random news site and don't change any polling settings, you have a default polling time of 100 minutes, on error 110 minutes, this is a bit of a joke.\r\nOtoh if you use it to get rss feeds from some random news site and don't change any polling settings, you have a default polling time of 100 minutes, on error 110 minutes, this is a bit of a joke. So making this additional thing configurable and extend the UI to edit it, add documentation to explain what it does and so on and so on I declare this obscure feature as not needed.","created_at":"2014-10-26T19:47:16Z","updated_at":"2015-09-23T08:57:50Z","closed_at":"2015-09-23T08:57:50Z","merged_at":"2015-09-23T08:57:50Z","merge_commit_sha":"4eff164903df81114cc635b34c0f1be8b04d17c7","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/plone/plone.app.portlets/pulls/44/commits","review_comments_url":"https://api.github.com/repos/plone/plone.app.portlets/pulls/44/comments","review_comment_url":"https://api.github.com/repos/plone/plone.app.portlets/pulls/comments{/number}","comments_url":"https://api.github.com/repos/plone/plone.app.portlets/issues/44/comments","statuses_url":"https://api.github.com/repos/plone/plone.app.portlets/statuses/18c14a9d9c5bcc49741bd6c6e863160392bc3c02","head":{"label":"do3cc:caching","ref":"caching","sha":"18c14a9d9c5bcc49741bd6c6e863160392bc3c02","user":{"login":"do3cc","id":147512,"avatar_url":"https://avatars.githubusercontent.com/u/147512?v=3","gravatar_id":"","url":"https://api.github.com/users/do3cc","html_url":"https://github.com/do3cc","followers_url":"https://api.github.com/users/do3cc/followers","following_url":"https://api.github.com/users/do3cc/following{/other_user}","gists_url":"https://api.github.com/users/do3cc/gists{/gist_id}","starred_url":"https://api.github.com/users/do3cc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/do3cc/subscriptions","organizations_url":"https://api.github.com/users/do3cc/orgs","repos_url":"https://api.github.com/users/do3cc/repos","events_url":"https://api.github.com/users/do3cc/events{/privacy}","received_events_url":"https://api.github.com/users/do3cc/received_events","type":"User","site_admin":false},"repo":{"id":25688375,"name":"plone.app.portlets","full_name":"do3cc/plone.app.portlets","owner":{"login":"do3cc","id":147512,"avatar_url":"https://avatars.githubusercontent.com/u/147512?v=3","gravatar_id":"","url":"https://api.github.com/users/do3cc","html_url":"https://github.com/do3cc","followers_url":"https://api.github.com/users/do3cc/followers","following_url":"https://api.github.com/users/do3cc/following{/other_user}","gists_url":"https://api.github.com/users/do3cc/gists{/gist_id}","starred_url":"https://api.github.com/users/do3cc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/do3cc/subscriptions","organizations_url":"https://api.github.com/users/do3cc/orgs","repos_url":"https://api.github.com/users/do3cc/repos","events_url":"https://api.github.com/users/do3cc/events{/privacy}","received_events_url":"https://api.github.com/users/do3cc/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/do3cc/plone.app.portlets","description":"provides a Plone-specific user interface for plone.portlets, as well as a standard set of portlets that ship with Plone","fork":true,"url":"https://api.github.com/repos/do3cc/plone.app.portlets","forks_url":"https://api.github.com/repos/do3cc/plone.app.portlets/forks","keys_url":"https://api.github.com/repos/do3cc/plone.app.portlets/keys{/key_id}","collaborators_url":"https://api.github.com/repos/do3cc/plone.app.portlets/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/do3cc/plone.app.portlets/teams","hooks_url":"https://api.github.com/repos/do3cc/plone.app.portlets/hooks","issue_events_url":"https://api.github.com/repos/do3cc/plone.app.portlets/issues/events{/number}","events_url":"https://api.github.com/repos/do3cc/plone.app.portlets/events","assignees_url":"https://api.github.com/repos/do3cc/plone.app.portlets/assignees{/user}","branches_url":"https://api.github.com/repos/do3cc/plone.app.portlets/branches{/branch}","tags_url":"https://api.github.com/repos/do3cc/plone.app.portlets/tags","blobs_url":"https://api.github.com/repos/do3cc/plone.app.portlets/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/do3cc/plone.app.portlets/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/do3cc/plone.app.portlets/git/refs{/sha}","trees_url":"https://api.github.com/repos/do3cc/plone.app.portlets/git/trees{/sha}","statuses_url":"https://api.github.com/repos/do3cc/plone.app.portlets/statuses/{sha}","languages_url":"https://api.github.com/repos/do3cc/plone.app.portlets/languages","stargazers_url":"https://api.github.com/repos/do3cc/plone.app.portlets/stargazers","contributors_url":"https://api.github.com/repos/do3cc/plone.app.portlets/contributors","subscribers_url":"https://api.github.com/repos/do3cc/plone.app.portlets/subscribers","subscription_url":"https://api.github.com/repos/do3cc/plone.app.portlets/subscription","commits_url":"https://api.github.com/repos/do3cc/plone.app.portlets/commits{/sha}","git_commits_url":"https://api.github.com/repos/do3cc/plone.app.portlets/git/commits{/sha}","comments_url":"https://api.github.com/repos/do3cc/plone.app.portlets/comments{/number}","issue_comment_url":"https://api.github.com/repos/do3cc/plone.app.portlets/issues/comments{/number}","contents_url":"https://api.github.com/repos/do3cc/plone.app.portlets/contents/{+path}","compare_url":"https://api.github.com/repos/do3cc/plone.app.portlets/compare/{base}...{head}","merges_url":"https://api.github.com/repos/do3cc/plone.app.portlets/merges","archive_url":"https://api.github.com/repos/do3cc/plone.app.portlets/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/do3cc/plone.app.portlets/downloads","issues_url":"https://api.github.com/repos/do3cc/plone.app.portlets/issues{/number}","pulls_url":"https://api.github.com/repos/do3cc/plone.app.portlets/pulls{/number}","milestones_url":"https://api.github.com/repos/do3cc/plone.app.portlets/milestones{/number}","notifications_url":"https://api.github.com/repos/do3cc/plone.app.portlets/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/do3cc/plone.app.portlets/labels{/name}","releases_url":"https://api.github.com/repos/do3cc/plone.app.portlets/releases{/id}","created_at":"2014-10-24T12:52:40Z","updated_at":"2014-10-24T13:03:01Z","pushed_at":"2015-09-22T07:45:02Z","git_url":"git://github.com/do3cc/plone.app.portlets.git","ssh_url":"git@github.com:do3cc/plone.app.portlets.git","clone_url":"https://github.com/do3cc/plone.app.portlets.git","svn_url":"https://github.com/do3cc/plone.app.portlets","homepage":"","size":2292,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"plone:master","ref":"master","sha":"7871ad8f9317dcaef686bed83d209b894d81d109","user":{"login":"plone","id":72034,"avatar_url":"https://avatars.githubusercontent.com/u/72034?v=3","gravatar_id":"","url":"https://api.github.com/users/plone","html_url":"https://github.com/plone","followers_url":"https://api.github.com/users/plone/followers","following_url":"https://api.github.com/users/plone/following{/other_user}","gists_url":"https://api.github.com/users/plone/gists{/gist_id}","starred_url":"https://api.github.com/users/plone/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/plone/subscriptions","organizations_url":"https://api.github.com/users/plone/orgs","repos_url":"https://api.github.com/users/plone/repos","events_url":"https://api.github.com/users/plone/events{/privacy}","received_events_url":"https://api.github.com/users/plone/received_events","type":"Organization","site_admin":false},"repo":{"id":2452743,"name":"plone.app.portlets","full_name":"plone/plone.app.portlets","owner":{"login":"plone","id":72034,"avatar_url":"https://avatars.githubusercontent.com/u/72034?v=3","gravatar_id":"","url":"https://api.github.com/users/plone","html_url":"https://github.com/plone","followers_url":"https://api.github.com/users/plone/followers","following_url":"https://api.github.com/users/plone/following{/other_user}","gists_url":"https://api.github.com/users/plone/gists{/gist_id}","starred_url":"https://api.github.com/users/plone/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/plone/subscriptions","organizations_url":"https://api.github.com/users/plone/orgs","repos_url":"https://api.github.com/users/plone/repos","events_url":"https://api.github.com/users/plone/events{/privacy}","received_events_url":"https://api.github.com/users/plone/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/plone/plone.app.portlets","description":"provides a Plone-specific user interface for plone.portlets, as well as a standard set of portlets that ship with Plone","fork":false,"url":"https://api.github.com/repos/plone/plone.app.portlets","forks_url":"https://api.github.com/repos/plone/plone.app.portlets/forks","keys_url":"https://api.github.com/repos/plone/plone.app.portlets/keys{/key_id}","collaborators_url":"https://api.github.com/repos/plone/plone.app.portlets/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/plone/plone.app.portlets/teams","hooks_url":"https://api.github.com/repos/plone/plone.app.portlets/hooks","issue_events_url":"https://api.github.com/repos/plone/plone.app.portlets/issues/events{/number}","events_url":"https://api.github.com/repos/plone/plone.app.portlets/events","assignees_url":"https://api.github.com/repos/plone/plone.app.portlets/assignees{/user}","branches_url":"https://api.github.com/repos/plone/plone.app.portlets/branches{/branch}","tags_url":"https://api.github.com/repos/plone/plone.app.portlets/tags","blobs_url":"https://api.github.com/repos/plone/plone.app.portlets/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/plone/plone.app.portlets/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/plone/plone.app.portlets/git/refs{/sha}","trees_url":"https://api.github.com/repos/plone/plone.app.portlets/git/trees{/sha}","statuses_url":"https://api.github.com/repos/plone/plone.app.portlets/statuses/{sha}","languages_url":"https://api.github.com/repos/plone/plone.app.portlets/languages","stargazers_url":"https://api.github.com/repos/plone/plone.app.portlets/stargazers","contributors_url":"https://api.github.com/repos/plone/plone.app.portlets/contributors","subscribers_url":"https://api.github.com/repos/plone/plone.app.portlets/subscribers","subscription_url":"https://api.github.com/repos/plone/plone.app.portlets/subscription","commits_url":"https://api.github.com/repos/plone/plone.app.portlets/commits{/sha}","git_commits_url":"https://api.github.com/repos/plone/plone.app.portlets/git/commits{/sha}","comments_url":"https://api.github.com/repos/plone/plone.app.portlets/comments{/number}","issue_comment_url":"https://api.github.com/repos/plone/plone.app.portlets/issues/comments{/number}","contents_url":"https://api.github.com/repos/plone/plone.app.portlets/contents/{+path}","compare_url":"https://api.github.com/repos/plone/plone.app.portlets/compare/{base}...{head}","merges_url":"https://api.github.com/repos/plone/plone.app.portlets/merges","archive_url":"https://api.github.com/repos/plone/plone.app.portlets/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/plone/plone.app.portlets/downloads","issues_url":"https://api.github.com/repos/plone/plone.app.portlets/issues{/number}","pulls_url":"https://api.github.com/repos/plone/plone.app.portlets/pulls{/number}","milestones_url":"https://api.github.com/repos/plone/plone.app.portlets/milestones{/number}","notifications_url":"https://api.github.com/repos/plone/plone.app.portlets/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/plone/plone.app.portlets/labels{/name}","releases_url":"https://api.github.com/repos/plone/plone.app.portlets/releases{/id}","created_at":"2011-09-25T01:39:02Z","updated_at":"2015-05-20T12:26:05Z","pushed_at":"2015-09-23T08:57:50Z","git_url":"git://github.com/plone/plone.app.portlets.git","ssh_url":"git@github.com:plone/plone.app.portlets.git","clone_url":"https://github.com/plone/plone.app.portlets.git","svn_url":"https://github.com/plone/plone.app.portlets","homepage":"","size":3253,"stargazers_count":13,"watchers_count":13,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":22,"mirror_url":null,"open_issues_count":1,"forks":22,"open_issues":1,"watchers":13,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/plone/plone.app.portlets/pulls/44"},"html":{"href":"https://github.com/plone/plone.app.portlets/pull/44"},"issue":{"href":"https://api.github.com/repos/plone/plone.app.portlets/issues/44"},"comments":{"href":"https://api.github.com/repos/plone/plone.app.portlets/issues/44/comments"},"review_comments":{"href":"https://api.github.com/repos/plone/plone.app.portlets/pulls/44/comments"},"review_comment":{"href":"https://api.github.com/repos/plone/plone.app.portlets/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/plone/plone.app.portlets/pulls/44/commits"},"statuses":{"href":"https://api.github.com/repos/plone/plone.app.portlets/statuses/18c14a9d9c5bcc49741bd6c6e863160392bc3c02"}},"merged":true,"mergeable":null,"mergeable_state":"unknown","merged_by":{"login":"jensens","id":157140,"avatar_url":"https://avatars.githubusercontent.com/u/157140?v=3","gravatar_id":"","url":"https://api.github.com/users/jensens","html_url":"https://github.com/jensens","followers_url":"https://api.github.com/users/jensens/followers","following_url":"https://api.github.com/users/jensens/following{/other_user}","gists_url":"https://api.github.com/users/jensens/gists{/gist_id}","starred_url":"https://api.github.com/users/jensens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jensens/subscriptions","organizations_url":"https://api.github.com/users/jensens/orgs","repos_url":"https://api.github.com/users/jensens/repos","events_url":"https://api.github.com/users/jensens/events{/privacy}","received_events_url":"https://api.github.com/users/jensens/received_events","type":"User","site_admin":false},"comments":17,"review_comments":0,"commits":3,"additions":47,"deletions":18,"changed_files":2}}