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's IPNS support broken #9641

Closed
RubenKelevra opened this issue May 5, 2020 · 1 comment
Closed

IPFS's IPNS support broken #9641

RubenKelevra opened this issue May 5, 2020 · 1 comment
Labels
closed/works-for-me feature/web3/ipfs priority/P4 Planned work. We expect to get to it "soon".

Comments

@RubenKelevra
Copy link

Description

When an IPNS link is opened, it will be forwarded to <IPNS-Hash>.ipns.localhost[:port].

It looks like the Brave Browser tries to resolve the .localhost domain instead of using it as a catch-all for 127.0.0.1 / ::1.

Steps to Reproduce

  1. open brave
  2. enable ipfs
  3. install a local ipfs daemon
  4. open a link like this

Actual result:

The ipfs plugin forwards the link correctly to http://<IPNS-Hash>.ipns.localhost[:port], but Brave seems to send the result to the DNS Server, instead of just using internal 127.0.0.1 / ::1 resolution.

Screenshot_20200505_233938

Expected result:

Open the page via the local webproxy on 127.0.0.1:80 / [::1]:80.

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave | 1.8.86 Chromium: 81.0.4044.129 (Official Build) unknown (64-bit)
-- | --
Revision | 3d71af9f5704a40b85806f4d08925db24605ba25-refs/branch-heads/4044@{#979}
OS | Linux
JavaScript | V8 8.1.307.31
Flash | (Disabled)
User Agent | Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.129 Safari/537.36
Command Line | /usr/lib/brave-bin/brave --enable-dom-distiller --disable-domain-reliability --no-pings --extension-content-verification=enforce_strict --extensions-install-verification=enforce --sync-url=https://no-thanks.invalid --enable-features=PasswordImport,WebUIDarkMode,AutoupgradeMixedContent,PassiveMixedContentWarning,MixedContentSiteSetting,SimplifyHttpsIndicator,DnsOverHttps --disable-features=AutofillServerCommunication,TextFragmentAnchor,VideoPlaybackQuality,LookalikeUrlNavigationSuggestionsUI,AllowPopupsDuringPageUnload,NotificationTriggers,AudioServiceOutOfProcess,SmsReceiver --flag-switches-begin --enable-features=PasswordImport,WebUIDarkMode,AutoupgradeMixedContent,PassiveMixedContentWarning,MixedContentSiteSetting,SimplifyHttpsIndicator,DnsOverHttps,BraveSync,DecodeJpeg420ImagesToYUV,DecodeLossyWebPImagesToYUV,ImpulseScrollAnimations,LegacyTLSEnforced,PaintHolding,PercentBasedScrolling,SyncSetupFriendlySettings --flag-switches-end --disable-webrtc-apm-in-audio-service
Executable Path | /usr/lib/brave-bin/brave

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?

no

  • Does the issue resolve itself when disabling Brave Rewards?

no

  • Is the issue reproducible on the latest version of Chrome Firefox?

Nope, firefox opens the link just fine.

@bbondy bbondy added this to Untriaged in IPFS Jun 11, 2020
@bbondy bbondy moved this from Untriaged to Backlog in IPFS Jun 11, 2020
@bbondy bbondy added the priority/P4 Planned work. We expect to get to it "soon". label Sep 2, 2020
@bbondy
Copy link
Member

bbondy commented Oct 30, 2020

I can't reproduce this problem, I think it was probably fixed at some point.
Otherwise it could be maybe automatic redirection causing problems on the IPFS Companion extension.

@bbondy bbondy closed this as completed Oct 30, 2020
IPFS automation moved this from Backlog to Done Oct 30, 2020
@bbondy bbondy removed this from Done in IPFS Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/works-for-me feature/web3/ipfs priority/P4 Planned work. We expect to get to it "soon".
Projects
None yet
Development

No branches or pull requests

3 participants