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

Unable to input/use https://cf-ipfs.com subdomain gateway #17648

Open
stephendonner opened this issue Aug 23, 2021 · 6 comments
Open

Unable to input/use https://cf-ipfs.com subdomain gateway #17648

stephendonner opened this issue Aug 23, 2021 · 6 comments

Comments

@stephendonner
Copy link

stephendonner commented Aug 23, 2021

Description

Unable to input/use https://cf-ipfs.com subdomain gateway.

Found while testing #16999.

Steps to Reproduce

  1. new profile
  2. launch Brave
  3. load brave://settings/ipfs
  4. click on the Change button
  5. enter https://cf-ipfs.com/ and click Submit

Actual result:

Invalid gateway address error message is returned.

Screen Shot 2021-08-23 at 4 42 10 PM

Expected result:

It should accept and use https://cf-ipfs.com as stable 1.28.105 does.

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.30.42 Chromium: 93.0.4577.51 (Official Build) nightly (x86_64)
Revision 762d21050e2da59930c784c09b134d0b0b148188-refs/branch-heads/4577@{#915}
OS macOS Version 11.5.2 (Build 20G95)

/cc @spylogsster @lidel

Version/Channel Information:

  • Can you reproduce this issue with the current release? No
  • Can you reproduce this issue with the beta channel? No
  • Can you reproduce this issue with the nightly channel? Yes
@spylogsster
Copy link

it shows captcha, cc @lidel
image

@lidel
Copy link

lidel commented Oct 18, 2021

iiuc options are:

  • cloudflare "safelists" gateway check done by Brave (request for a small CID), or..
  • Brave opens captcha screen in a new tab, and repeats the test after page navigates away to the desired payload

@thibault thoughts from Cloudflare side, which option you prefer?
I think safelisting the test CID is better UX but lmk if it is feasible on your end (@spylogsster can provide you with the CID we use for gateway test)

@thibault
Copy link

Sorry, wrong @thibault

@stephendonner
Copy link
Author

Also https://cf-ipfs.com/ipns

origin-isolation

@srirambv
Copy link
Contributor

srirambv commented Dec 5, 2022

This seems to work fine on 1.48.24 with Kubo 0.16.0 and 1.47.107 with Kubo 0.17.0

1.48.24 with Kubo 0.16.0 1.47.107 with Kubo 0.17.0
17648.-.0.16.0.mov
17648.-.0.17.0.mov

cc: @lidel @spylogsster

@lidel
Copy link

lidel commented Dec 6, 2022

@cypt4 thoughts on improving error reporting here?

Are we able to detect Cloudflare's captcha and open test request in new tab? Or maybe show error suggesting https://chrome.google.com/webstore/detail/privacy-pass/ajhmfdgkijocedmfjonnpjfojldioehi?
Sounds feasible or too complex?

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

No branches or pull requests

5 participants