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.

Sign up for GitHub

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

Jump to bottom

ipfs add <link> forces a multihash #325

Closed
jbenet opened this issue Nov 13, 2014 · 4 comments
Closed

ipfs add <link> forces a multihash #325

jbenet opened this issue Nov 13, 2014 · 4 comments
Milestone
α

Comments

@jbenet
Copy link
Member

jbenet commented Nov 13, 2014

this should work:

% ipfs2 cat /ipfs/QmaCpDMGvV2BGHeYERUEnRQAwe3N8SzbUtfsmvsqQLuvuJ
Error: multihash length inconsistent: &{124  130 [210]}
@jbenet jbenet added this to the merge-commands milestone Nov 13, 2014
@mappum
Copy link
Contributor

mappum commented Nov 14, 2014

The paths are being directly passed to Resolver.ResolvePath(), maybe the change should happen there? It could just test if the string starts with /ipfs/ and if so remove the prefix.

@jbenet jbenet removed this from the merge-commands milestone Nov 14, 2014
@jbenet
Copy link
Member Author

jbenet commented Jan 8, 2015

This is still a problem. must be fixed for alpha.

@jbenet
Copy link
Member Author

jbenet commented Jan 8, 2015 

> ipfs add Makefile
added QmVE4ayZnqn3dbP6kTgkANrtXbKogVCaHVDnVnmEgURBmC Makefile
> ipfs cat /ipfs/QmVE4ayZnqn3dbP6kTgkANrtXbKogVCaHVDnVnmEgURBmC
Error: multihash length inconsistent: &{124  130 [210]}

@jbenet jbenet added this to the α milestone Jan 8, 2015
@btc
Copy link
Contributor

btc commented Jan 10, 2015

appears to be fixed as of 670e693

@jbenet re-open if I'm mistaken

@btc btc closed this as completed Jan 10, 2015
ariescodescream pushed a commit to ariescodescream/go-ipfs that referenced this issue Oct 23, 2021
@Stebalien
Merge pull request ipfs#325 from libp2p/fix/gc
5b6bc7e 
fix(providers): gc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
α
Development

No branches or pull requests
3 participants
@jbenet @mappum @btc