snarfed / bridgy

📣 Connects your web site to social media. Likes, retweets, mentions, cross-posting, and more...
https://brid.gy
Creative Commons Zero v1.0 Universal
710 stars 52 forks source link

Blogger domain localisation breaks webmentions via bridgy #805

Closed kevinmarks closed 6 years ago

kevinmarks commented 6 years ago

If I go to a blogger post while in the uk, instead of: http://epeus.blogspot.com/2008/01/urls-are-people-too.html

I get redirected to: http://epeus.blogspot.co.uk/2008/01/urls-are-people-too.html

When I link to this post eg from https://mastodon.social/@kevinmarks/99801691700756768

and then send a webmention using mention.tech things get confused. If I look up the .co.uk version, the webmention validates for mention.tech, but then bridgy gives a 400:

SendMention POST to https://www.brid.gy/webmention/blogger result.content { (/base/data/home/apps/s~mention-tech/1.408033659792851681/main.py:240)
  "error": "Could not find Blogger account for epeus.blogspot.co.uk. Is it registered with Bridgy?"
}

If I change the webmention target to the .com version, then the link is not found in the source post.

Not sure how to fix this - seems bridgy would need knowledge of blogger's idiosyncrasies here.

kevinmarks commented 6 years ago

There is a rel="canonical" link:

<link href='http://epeus.blogspot.com/2008/01/urls-are-people-too.html' rel='canonical'/>
kevinmarks commented 6 years ago

in the uk (presumably geoip) I get a 302 on the canonical url:

curl --head http://epeus.blogspot.com/2008/01/urls-are-people-too.html
HTTP/1.1 302 Moved Temporarily
Location: http://epeus.blogspot.co.uk/2008/01/urls-are-people-too.html
snarfed commented 6 years ago

thanks for filing, and for the useful details! and sorry for the trouble.

bridgy already encodes lots of silo peculiarities, so I'd be fine with adding this one. any chance you know where they list their domain logic? if not i can see what i can find.

snarfed commented 6 years ago

ah! if rel canonical is consistent, that's perfect, and easy. I'll try it.

snarfed commented 6 years ago

(btw @kevinmarks, just fyi, your blogger auth on bridgy expired at some point: https://brid.gy/blogger/3200930 )

snarfed commented 6 years ago

ok, i think this is fixed. feel free to try again once you're re-authed!

kevinmarks commented 6 years ago

that did work, though the comment preview didn't come out that well - any idea why it got ellipsised with so little text? http://epeus.blogspot.co.uk/2008/01/urls-are-people-too.html

On Fri, Apr 6, 2018 at 7:42 AM, Ryan Barrett notifications@github.com wrote:

ok, i think this is fixed. feel free to try again once you're re-authed!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/snarfed/bridgy/issues/805#issuecomment-379163083, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGCwKRJJrZ1uy9yRHW3ax3K3-Kn7DCHks5tlw5JgaJpZM4TG-e8 .

snarfed commented 6 years ago

no good reason! thanks for the nudge, i'll increase the snippet length.

kevinmarks commented 6 years ago

Was it measuring it in HTML rather than rendered text?

On Mon, 9 Apr 2018, 21:34 Ryan Barrett, notifications@github.com wrote:

no good reason! thanks for the nudge, i'll increase the snippet length.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/snarfed/bridgy/issues/805#issuecomment-379885155, or mute the thread https://github.com/notifications/unsubscribe-auth/AAGCwObVs4Z8a027GHlisAf3MSI8_CGGks5tm8W3gaJpZM4TG-e8 .

snarfed commented 6 years ago

nah, just 30 char limit for no good reason.