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

Enable Gateway on localhost IPv6 (ip4+ip6 by default) #5905

Open
Tracked by #10000
lidel opened this issue Jan 8, 2019 · 7 comments
Open
Tracked by #10000

Enable Gateway on localhost IPv6 (ip4+ip6 by default) #5905

lidel opened this issue Jan 8, 2019 · 7 comments
Labels
kind/enhancement A net-new feature or improvement to an existing feature topic/gateway Topic gateway

Comments

@lidel
Copy link
Member

lidel commented Jan 8, 2019

Version information:

0.4.18

Type:

enhancement

Description:

What? How do we feel about migrating default Gateway config to listen on ipv6 as well?
For now, this can be enabled manually via:

$ ipfs config --json Addresses.Gateway '["/ip4/127.0.0.1/tcp/8080","/ip6/::1/tcp/8080"]'

Why? It is a cosmetic change that enables much shorter URLs for local gateway and shift focus towards IPFS paths:

2019-01-08-125329_758x575_scrot

How?

Current default:

$ ipfs config --json Addresses.Gateway
/ip4/127.0.0.1/tcp/8080

Proposed change (arrays are supported since ipfs/go-ipfs-config#11):

$ ipfs config --json Addresses.Gateway
[
  "/ip4/127.0.0.1/tcp/8080",
  "/ip6/::1/tcp/8080"
]

Additional notes:

@Stebalien
Copy link
Member

We should definitely listen on both. Mind making a PR?

@Stebalien Stebalien added kind/enhancement A net-new feature or improvement to an existing feature topic/gateway Topic gateway labels Jan 8, 2019
@alanshaw
Copy link
Member

...also to JS IPFS 🙏

@da2x
Copy link
Contributor

da2x commented Feb 27, 2019

For what it’s worth, I’ve been running my local gateway on [::1] for almost a year with no issues.

@Kubuxu
Copy link
Member

Kubuxu commented Feb 27, 2019

@Stebalien We can include it in upcoming migration, what do you think?

@Stebalien
Copy link
Member

SGTM (iff the user is already listening on 127.0.0.1).

@Winterhuman
Copy link
Contributor

@lidel Any update on this?

@lidel
Copy link
Member Author

lidel commented Apr 13, 2022

No update. Browser use case ended up having a better solution in the form of subdomain gateway on {cid}.ipfs.localhost:8080 where each CID gets its own Origin sandbox, so adding ip6 listener lost its utility.

Something we would still want to do is to detect when the system does not support ip4 and then use ip6 instead, but this is low priority since we had no user reports that this is a real issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A net-new feature or improvement to an existing feature topic/gateway Topic gateway
Projects
None yet
Development

No branches or pull requests

6 participants