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

IPFS Companion not switching to local Gateway even when X-Ipfs-Path is provided #953

Closed
simonfoilen opened this issue Dec 27, 2020 · 4 comments
Assignees
Labels
kind/bug A bug in existing code (including security flaws)

Comments

@simonfoilen
Copy link

simonfoilen commented Dec 27, 2020

Browser Setup

  • OS: Linux Ubuntu 20.04.1 LTS
  • Browser: Chrome Version 87.0.4280.88 (Official Build) (64-bit)
  • IPFS Companion Version: 2.16.0
  • IPFS Companion has been reset to default preferences

Working for main site

Steps:

Issue with a custom site

Steps to repreduce

Steps:

Custom site setup

[OK] DNSLink

dig test.cloud.foilen.com TXT

    ;; ANSWER SECTION:
    test.cloud.foilen.com.	1	IN	TXT	"dnslink=/ipns/k51qzi5uqu5dhuj92m8egzrbx6e0apodpebvs7y4fqe1rc6rxb1hsiwgovl94o"

[OK] X-Ipfs-Path Header

curl -v https://test.cloud.foilen.com 2>&1 | grep X-Ipfs-Path
    < X-Ipfs-Path: /ipns/test.cloud.foilen.com/

Misc

Also, using my local Gateway directly is working


Could you let me know if my setup is missing something or fix the Chrome extension if there is a bug with it?

thanks

@simonfoilen simonfoilen added the need/triage Needs initial labeling and prioritization label Dec 27, 2020
@jessicaschilling
Copy link
Contributor

jessicaschilling commented Dec 27, 2020

Thank you for the detailed report. Many of the core team are at reduced capacity for the holidays, so it's suggested to open this as a post in the forums at discuss.ipfs.io so your question can be more easily seen by the community as a whole. Thank you!

@ipfs ipfs deleted a comment from welcome bot Dec 27, 2020
@lidel
Copy link
Member

lidel commented Dec 28, 2020

Hi @simon, thank you for bringing this issue up.
I can confirm this is a bug related to parsing /ipns/ in DNSLink records, we are tracking it in: ipfs/kubo#6454 (comment)

TLDR workaround: until it is fixed upstream in go-ipfs, you need to use immutable paths starting with /ipfs/{cid}/

@simonfoilen
Copy link
Author

simonfoilen commented Dec 28, 2020

I confirm the workaround works. Thanks

@jessicaschilling jessicaschilling removed the need/triage Needs initial labeling and prioritization label Jan 4, 2021
@lidel
Copy link
Member

lidel commented Jan 18, 2021

I believe this is fixed in v2.17.1 :-)

@lidel lidel closed this as completed Jan 18, 2021
@lidel lidel added the kind/bug A bug in existing code (including security flaws) label Jan 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug A bug in existing code (including security flaws)
Projects
None yet
Development

No branches or pull requests

3 participants