Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

One link isn't getting picked up #37

Open
leo60228 opened this issue Feb 5, 2021 · 7 comments
Open

One link isn't getting picked up #37

leo60228 opened this issue Feb 5, 2021 · 7 comments

Comments

@leo60228
Copy link

leo60228 commented Feb 5, 2021

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
Copy link
Owner

remy commented Feb 5, 2021

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
Copy link
Author

leo60228 commented Feb 5, 2021

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
Copy link
Author

leo60228 commented Feb 5, 2021

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?

leo60228 added a commit to leo60228/leoblog that referenced this issue Feb 5, 2021
@remy
Copy link
Owner

remy commented Feb 6, 2021

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
Copy link
Author

leo60228 commented Feb 6, 2021

@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
Copy link

snarfed commented Feb 7, 2021

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants