Make feed PubSubHubbub enabled

[Michele Azzolari]

I created a branch [1] to make feed PubSubHubbub enabled.

To enable it add these 2 lines to your mediagoblin_local.ini

push_enabled = True push_url = 'http://pubsubhubbub.appspot.com/'

[1] [https://gitorious.org/_{macno/mediagoblin/macnos-mediagoblin/commits/pushatom](https://gitorious.org/}macno/mediagoblin/macnos-mediagoblin/commits/pushatom)

[Elrond]

Stupid question: Why not enable just/when/if the url is set? Two config settings feels a bit "too much"?

[Michele Azzolari]

Removed one setting. If push_url is set, we use it

[https://gitorious.org/_{macno/mediagoblin/macnos-mediagoblin/commit/7f251b037bd5207652ca73f556e90b9633786a3c](https://gitorious.org/}macno/mediagoblin/macnos-mediagoblin/commit/7f251b037bd5207652ca73f556e90b9633786a3c)

[Elrond]

That looks good! I'll try to merge this tomorrow.

[Elrond]

Hi Michele,

I merged your tree (so please do a git merge upstream/master or equivalent).

But (and that's the reason I'm not closing the bug):

If the remote server is broken (you can easily test this using http://localhost/no-such-file, as a push_urls setting) one gets a big fat error page (urllib2.HTTPError exception).

I am currently unsure what is the best option for this.

My vote is on "catch the exception and add the stringified version of the Exception as a message", so the users gets it. Maybe not the best, but our current userbase will surely know what to do with such a strange thing.

[Michele Azzolari]

I pushed the changes to my branch, check it out

[Elrond]

merged! Great work!

[Will Kahn-Greene]

The original url for this bug was ​http://bugs.foocorp.net/issues/736 .
Relations:
#311: related