pubsubhubbub / PubSubHubbub

The PubSubHubbub protocol specification.
http://pubsubhubbub.github.io/PubSubHubbub
603 stars 122 forks source link

Allow "sync" verification #9

Closed evanp closed 10 years ago

evanp commented 12 years ago

PuSH 0.3 allowed a "sync" verification mechanism.

It's not clear to me why we pulled this out.

jobetudiant commented 12 years ago

Because it can't work if we want the feed publisher's approval. If you're a subscriber and want 'sync', but the feed is protected, you won't get an verification right away, as the hub needs to ask the publisher.

In my experience, having 2 different modes made things a lot messier for a lot of people. I'm in favor of using the mode that allows everyone to get stuff done, rather than 2 modes where people pick the wrong one and eventually figure out that the one they picked didn't work.

superfeedr commented 12 years ago

Dammit! Wrong Github account... but the previous comment was from me!

evanp commented 12 years ago

Seems like a negotiation process at subscription time, then.

if (topic requires async) {
    if (subscriber only does sync) {
        hub returns 400 failure;
    } else {
        do async;
  }
} else {
    if (subscriber only does sync) {
        do sync;
    } else if (subscriber only does async) {
        do async;
    } else {
        do either!;
    }
}
superfeedr commented 12 years ago

Complexity... and weirdness. Why would we let subscribers assume they can do "sync" when they can't?

superfeedr commented 12 years ago

Another problem with the sycn approach is that it the subscriber cannot know before actually subscribing whether the feed requires a publisher validation... and if it does, then, there is no way this can be sync'ed, because the publisher may take time (at least user authorization) before it validates the subscription...