PubSubHubbub #831
Merged
PubSubHubbub #831
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
FreshRSS#312 Requires setting base_url in config.php. Currently using the filesystem (no change to the database)
FreshRSS#312 Using a white list limited to http://push-pub.appspot.com/feed for alpha testing.
Closed
|
I have done a special effort to make only very few changes to the FreshRSS code when adding PubSubHubbub, so the pull request is relatively small compared to the impact of this new feature. |
|
+1 |
FreshRSS#312 Show whether PubSubHubbub is enabled in the Web interface of feed configuration. When PubSubHubbub is used, do not pull refresh so often (hard-coded to max once per 24h for now). Improved logic for lease renewal, and some detection of lease problems. Updated read-me and changelog.
| @@ -84,6 +84,7 @@ | |||
| 'articles_per_page' => 'Počet článků na stranu', | |||
| 'auto_load_more' => 'Načítat další články dole na stránce', | |||
| 'auto_remove_article' => 'Po přečtení články schovat', | |||
| 'mark_updated_article_unread' => 'Označte aktualizované položky jako nepřečtené', | |||
| @@ -37,6 +37,7 @@ | |||
| 'url' => 'Feed-URL', | |||
| 'validator' => 'Überprüfen Sie die Gültigkeit des Feeds', | |||
| 'website' => 'Webseiten-URL', | |||
| 'pubsubhubbub' => 'Sofortige Benachrichtigung mit PubSubHubbub', | |||
There was a problem hiding this comment.
@thomase1993 Please check
There was a problem hiding this comment.
Je préfère 'Sofortbenachrichtigung mit PubSubHubbub' ou 'Sofortbenachrichtigung durch PubSubHubbub'
There was a problem hiding this comment.
Merci, c'est fait Alkarex@d3af903 :-)
Planned for version beta 1.1.2 FreshRSS#312
Alkarex
added a commit
that referenced
this pull request
May 16, 2015
Several spelling mistakes corrected. Slight rewording of some sentences. Cherry-picked from PubSubHubbub #831 (it should have been a different branch) Alkarex@3adab4b
Do not pull refresh feeds that are PubSubHubbub too often during cron refresh. And more debugging info during the test phase. FreshRSS#831
|
I suggest leaving this pull request open till after we release beta 1.1.1, so that we can use the current /dev to release beta 1.1.1. We can target beta 1.1.2 next month for PubSubHubbub. |
Do not assume that PubSubHubbub works until the first successul push FreshRSS#312 (comment)
Closed
Alkarex
referenced
this pull request
in Alkarex/FreshRSS
May 23, 2015
FreshRSS#848 Corrections in Minz (HTTP_HOST was not sanitized, getURI() was never used and not working anyway with absolute base_url) $this->url was not defined in rss.phtml
The tests so far are good. Ready to test more broadly. FreshRSS#831 FreshRSS#312
|
Merging the PubSubHubbub code to start testing in /dev towards next FreshRSS version 1.1.2 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#312
http://blog.superfeedr.com/howto-pubsubhubbub/