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

Open using IPFS button is missing sometimes #20340

Closed
stephendonner opened this issue Jan 5, 2022 · 3 comments · Fixed by brave/brave-core#11790
Closed

Open using IPFS button is missing sometimes #20340

stephendonner opened this issue Jan 5, 2022 · 3 comments · Fixed by brave/brave-core#11790
Assignees

Comments

@stephendonner
Copy link

Description

Open using IPFS button is missing sometimes

Steps to Reproduce

  1. new profile
  2. launch Brave
  3. load https://bafybeicgmdpvw4duutrmdxl4a7gc52sxyuk7nz5gby77afwdteh3jc5bqa.ipfs.dweb.link/wiki/Mars.html
  4. click on Use a local node on the interstitial page
  5. wait for page to fully load
  6. look at the URL bar

Actual result:

Screen Shot 2022-01-05 at 10 35 58 AM

There's no Open using IPFS button

Expected result:

As there is on https://en.wikipedia-on-ipfs.org/wiki/, I'd expect there to be an Open using IPFS button, since the page does have x-ipfs-path: /ipfs/bafybeicgmdpvw4duutrmdxl4a7gc52sxyuk7nz5gby77afwdteh3jc5bqa/wiki/Mars.html

Screen Shot 2022-01-05 at 10 40 05 AM

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.33.106 Chromium: 96.0.4664.110 (Official Build) (arm64)
Revision d5ef0e8214bc14c9b5bbf69a1515e431394c62a6-refs/branch-heads/4664@{#1283}
OS macOS Version 11.6 (Build 20G165)

Reproduced this issue with both go-ipfs version 0.10.0 and 0.11.0.

cc @spylogsster and @lidel

@lidel
Copy link

lidel commented Jan 10, 2022

I believe this specific case should work even when HTTP header is not present. The header is a useful hint on DNSLink pages to trigger TXT lookup, but here the user agent (Brave) should detect /ipfs/{valid-cid} in URL – that is enough information to display the button.

@autonome
Copy link

So does that mean it's a regression? I don't remember this working spottily in previous versions.

@stephendonner
Copy link
Author

Verified PASSED using

Brave 1.36.30 Chromium: 97.0.4692.71 (Official Build) nightly (x86_64)
Revision adefa7837d02a07a604c1e6eff0b3a09422ab88d-refs/branch-heads/4692@{#1247}
OS macOS Version 12.2 (Build 21D5025f)

Confirmed that with IPFS installed as a local node, loading https://bafybeicgmdpvw4duutrmdxl4a7gc52sxyuk7nz5gby77afwdteh3jc5bqa.ipfs.dweb.link/wiki/Mars.html yields the IPFS button (we shortened it as we've done with Tor).

Screen Shot 2022-01-11 at 1 54 21 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants