remy / wm

webmention check service
https://webmention.app
103 stars 17 forks source link

One link isn't getting picked up #37

Open leo60228 opened 3 years ago

leo60228 commented 3 years ago

I'm using webmention.app with https://leo60228.space/atom.xml. It picks up https://leo60228.tumblr.com and https://webmention.io in "Automatic syndication and cross-platform interactions with blogs." However, it doesn't pick up https://brid.gy/publish/mastodon. Even more oddly, this is picked up for "IndieWeb."

remy commented 3 years ago
Screenshot 2021-02-05 at 23 46 55

I'm seeing this ☝️ when I posted your feel url into the site (which should be the same as the command line if you're using it).

I'm not sure I quite follow what you're expecting to see (I just need a bit more of a clear idea) - what source and target is missing (and do you know which post in your feed it relates to)?

leo60228 commented 3 years ago

I'm expecting this:

  1. source= https://leo60228.space/automatic-syndication-and-cross-platform-interactions-with-blogs/
    target= https://leo60228.tumblr.com
  2. source= https://leo60228.space/automatic-syndication-and-cross-platform-interactions-with-blogs/
    target= https://webmention.io
  3. source= https://leo60228.space/automatic-syndication-and-cross-platform-interactions-with-blogs/
    target= https://brid.gy/publish/mastodon
  4. source= https://leo60228.space/indieweb/
    target= https://brid.gy/publish/mastodon
leo60228 commented 3 years ago

Testing locally, it works if I remove the link to https://brid.gy. Is it maybe caching somewhere that the brid.gy domain doesn't support Webmentions?

leo60228 commented 3 years ago

It seems like it: https://github.com/remy/wm/blob/master/lib/endpoint.js#L8-L20

Maybe a lack of Webmention endpoint shouldn't be cached?

remy commented 3 years ago

I'll have to pull in your atom feed as a fixture and debug what's going on. It's been a while since I've put my head in this code, so it takes a little while to get back in to it.

leo60228 commented 3 years ago

@remy I've implemented a workaround in the live site. Here's the atom.xml that still causes the issue: https://gist.github.com/b077279b0bef8366208d3fca65e63665

snarfed commented 3 years ago

In case it helps, Bridgy publish URLs like https://brid.gy/publish/mastodon do advertise a webmention endpoint, but the home page https://brid.gy/ doesn't. If webmention.app is caching webmention endpoints by domain instead of full URL, then yeah, you're going to have a bad time...but I feel your pain. Technically the spec doesn't allow any caching (lots of discussion here), even for full URLs, but Bridgy does the same caching by domain in order to scale. I ran into the same issue with some home pages not advertising endpoints, so I made Bridgy special case home pages. Just a band-aid, but definitely helped.