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

e-content mandatory but it should not #27

Open
hyde-stevenson opened this issue Jun 3, 2020 · 4 comments · May be fixed by #57
Open

e-content mandatory but it should not #27

hyde-stevenson opened this issue Jun 3, 2020 · 4 comments · May be fixed by #57
Labels
help wanted Extra attention is needed

Comments

@hyde-stevenson
Copy link

Hello,

I spent some times to understand why I couldn't get the webmention sent.
It looks like your parser validates the content only if an e-content is present.
From the documentation, that's not mandatory. And a small p-summary by example should be enough.

Could you check that on your side ?

Regards,

@remy
Copy link
Owner

remy commented Jun 3, 2020

I'll need a URL that you're asking the webmention.app to query.

@hyde-stevenson
Copy link
Author

I actually fixed it adding an e-content but the specification of microformats2 doesn't require it.

@aciccarello
Copy link

I encountered a similar issue with bridgy-fed. The bridgy fed docs specifically recommend putting the link outside e-content. But webmention.app doesn't see the bridgy-fed link in this scenario. I wonder if this limitation should be reconsidered.

#44 #50 also seem to be related to this confusion

@remy
Copy link
Owner

remy commented Feb 16, 2023

The test suite for webmention.app is fairly solid, if you can provide me a sample of html that would cause this issue, a fix could be developed against it.

@remy remy added the help wanted Extra attention is needed label Feb 16, 2023
aciccarello added a commit to aciccarello/wm that referenced this issue Feb 24, 2023
aciccarello added a commit to aciccarello/wm that referenced this issue Feb 24, 2023
@aciccarello aciccarello linked a pull request Feb 24, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
3 participants