idunnololz,
@idunnololz@lemmy.world avatar

Thanks for reporting this. I’ll look into the issue later today.

idunnololz,
@idunnololz@lemmy.world avatar

Ah I found the issue. The link you posted is https://next.lemm.ee/post/27522337 however the instance is not next.lemm.ee. The instance is lemm.ee. This is a bit challenging to handle since nothing prevents instances from being hosted on a subdomain so Summit has no idea if next.lemm.ee is the instance or lemm.ee. I’ll look into how I could resolve this issue going forward.

sunaurus,

Hey! Do I understand correctly that your app is trying to fetch any /post/<id> directly from its source instance API? If so, I have a few ideas:

  1. If you detect anything that looks like a Lemmy post URL (<host>/post/<id>), you could first try to make a request to <host>/version, and only treat the post as a Lemmy post if you get a JSON response from that endpoint where .software.name === ‘lemmy’. Otherwise, open that URL in the browser.

  2. The above will already “fix” the issue, but custom frontends on subdomains is actually quite common, so if you want to handle those links without resorting to a browser, then: in cases where <host>/version is not a Lemmy response, but <host> includes a subdomain, you could repeat the same logic from step 1 for the parent domain as well. If you detect a Lemmy API at the parent domain, then just use that to fetch the post.

idunnololz,
@idunnololz@lemmy.world avatar

Thank you for the suggestion! Not to worry I’ve already implemented a fix. It will go out in the next release.

  • All
  • Subscribed
  • Moderated
  • Favorites
  • summit@lemmy.world
  • osvaldo12
  • magazineikmin
  • thenastyranch
  • Youngstown
  • mdbf
  • everett
  • slotface
  • khanakhh
  • cisconetworking
  • rosin
  • kavyap
  • ethstaker
  • InstantRegret
  • DreamBathrooms
  • megavids
  • ngwrru68w68
  • Durango
  • tester
  • tacticalgear
  • Leos
  • GTA5RPClips
  • modclub
  • cubers
  • provamag3
  • anitta
  • normalnudes
  • JUstTest
  • lostlight
  • All magazines