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 doesn't accept multibase IPNS addresses #3977

Closed
Stebalien opened this issue Jun 13, 2017 · 6 comments
Closed

IPFS doesn't accept multibase IPNS addresses #3977

Stebalien opened this issue Jun 13, 2017 · 6 comments
Labels
need/analysis Needs further analysis before proceeding

Comments

@Stebalien
Copy link
Member

Version information:

go-ipfs version: 0.4.9-8ec0814a
Repo version: 5
System version: amd64/linux
Golang version: go1.8.3

Type:

Enhancement?

Severity:

Low

Description:

IPFS doesn't accept multibase IPNS addresses. For example, try fetching wikipedia with a multibase prefix:

ipfs cat /ipns/zQmdJiuMWp2FxyaerfLrtdLF6Nr1EWpL7dPAxA9oKSPYYgV/wiki/index.html
@kevina kevina self-assigned this Aug 30, 2017
@whyrusleeping
Copy link
Member

hrm... Should we add a cid multicodec for ipns?

@Stebalien
Copy link
Member Author

Lately, I've been thinking of generalizing CIDv0: multiformats/cid#12

But yes, the alternative is to just say Qm... in /ipns/Qm... is not a CID, SomethingElse in /ipns/SomethingElse is.

@kevina kevina added the need/analysis Needs further analysis before proceeding label Nov 28, 2017
@kevina
Copy link
Contributor

kevina commented Nov 28, 2017

Note: There is some debate is CidV0 hashes should even be allowed a multibase prefix so I'm not sure the correct thing to do here.

@Stebalien
Copy link
Member Author

@kevina at the moment, IPNS addresses aren't CIDs. They're just raw multihashes (that point to special protocol buffers, not IPLD nodes).

@reardenlife
Copy link

@whyrusleeping

"hrm... Should we add a cid multicodec for ipns?"

Was it ever implemented -- the CIDv1 for IPNS?

@lidel
Copy link
Member

lidel commented Sep 29, 2021

Done. IPNS support for multibase names being CIDv1 with libp2p-key codec was implemented around the time #7441 landed 🐈
See https://github.com/libp2p/specs/blob/master/RFC/0001-text-peerid-cid.md

@lidel lidel closed this as completed Sep 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need/analysis Needs further analysis before proceeding
Projects
None yet
Development

No branches or pull requests

5 participants