If caching is enabled, keep track of already-sent pings

PlaidWeb / Pushl

Push notification adapter for feeds

MIT License

30 stars 0 forks source link

If caching is enabled, keep track of already-sent pings #15

Closed fluffy-critter closed 5 years ago

fluffy-critter commented 5 years ago

At least in the case of Pingback, most sites do not seem to appreciate getting the same ping multiple times (as opposed to Webmention which makes that part of the spec).

So, if persistent storage is enabled, pushl should only send pingback pings which haven't been sent before.

fluffy-critter commented 5 years ago

really what it should do is track what the last pinglist was and only send the set xor of it, for webmentions as well.

~~except in the special case of 410 GONE in which case it should send for all prior pings.~~ oh yeah 410 GONE means the ping list will be empty so it gets this automatically

fluffy-critter commented 5 years ago

Trivial fix in 1d48bd5b260468b0e15a707757e0c8021dee92ac