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

self-reply logic in Protocol.targets not consistently using domains for Web ids #1624

Open
gerbenjacobs opened this issue Dec 14, 2024 · 1 comment
Labels
bug User-facing breakage and reliability issues within Bridgy Fed.

Comments

@gerbenjacobs
Copy link

I'm using web -> fed.brid.gy and, while not documented on /docs I figured I would try to reply to my own note using u-in-reply-to microformat in order to thread my texts.

Unfortunately it didn't work, while it itself works as a webmention and even, to my surprise, works on Bluesky.

Image

(Shortened) HTML for reply:

<article class="h-entry">
        <p>
            <small>In reply to: <a class="u-in-reply-to" href="https://gerben.dev/notes/2b7c13dd25a4c978e10267661216ae4b">
            https://gerben.dev/notes/2b7c13dd25a4c978e10267661216ae4b</a></small>
        </p>
        <div class="e-content">
            <p>I did however easily add Markdown support to my site.</p>
            <p>.. and this previous post helped me bug fix.</p>
        </div>
    <a class="u-bridgy-fed" href="https://fed.brid.gy/" hidden="hidden"></a>
</article>

On the fediverse it wasn't sent as a thread, it wasn't even sent as a regular post/note/text.

Bluesky thread: https://bsky.app/profile/gerben.dev.web.brid.gy/post/3ldc7v7uszri2
Original note: https://gerben.dev/notes/2b7c13dd25a4c978e10267661216ae4b (the webmention was manually sent)
Reply note: https://gerben.dev/replies/50be49821711d3d87c1d5f20e44aacb6

@snarfed
Copy link
Owner

snarfed commented Dec 15, 2024

Thanks for the detailed sleuthing and report! Looks like our self-reply logic for these situations is a bit flaky and got this wrong. Will fix.

For my own notes here, this is a domain vs homepage URL id mismatch on owner ids. Specifically, in Protocol.targets, owner here ended up as https://gerben.dev/:

owner = as1.get_owner(obj.as1)

...but as1.get_owner(orig_obj.as1) here was gerben.dev:

if id in in_reply_tos and owner == as1.get_owner(orig_obj.as1):

@snarfed snarfed changed the title feature: Support u-in-reply-to to websites, to allow threading self-reply logic is Protocol.targets not consistently using domains for Web ids Dec 15, 2024
@snarfed snarfed changed the title self-reply logic is Protocol.targets not consistently using domains for Web ids self-reply logic in Protocol.targets not consistently using domains for Web ids Dec 15, 2024
@snarfed snarfed added now and removed now labels Dec 15, 2024
@Tamschi Tamschi added the bug User-facing breakage and reliability issues within Bridgy Fed. label Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug User-facing breakage and reliability issues within Bridgy Fed.
Projects
None yet
Development

No branches or pull requests

3 participants