Skip to content

fix: improved error message on broken CIDv0#7807

Merged
aschmahmann merged 1 commit intomasterfrom
fix/broken-cidv0-ux
Jan 14, 2021
Merged

fix: improved error message on broken CIDv0#7807
aschmahmann merged 1 commit intomasterfrom
fix/broken-cidv0-ux

Conversation

@lidel
Copy link
Member

@lidel lidel commented Dec 7, 2020

Closes #7792

$ ipfs files stat /ipfs/qmbwqxbekc3p8tqskc98xmwnzrzdtrlmimpl8wbutgsmnr
Error: invalid path "/ipfs/qmbwqxbekc3p8tqskc98xmwnzrzdtrlmimpl8wbutgsmnr": invalid CID: selected encoding not supported (possible lowercased CIDv0; consider converting to a case-agnostic CIDv1, such as base32)

Would be cool to squeeze this in go-ipfs 0.8.0.

@lidel lidel requested a review from aschmahmann December 7, 2020 18:01
@lidel lidel added the topic/cidv1b32 Topic cidv1b32 label Dec 7, 2020
@lidel lidel force-pushed the fix/broken-cidv0-ux branch from 17616f9 to 2cb958d Compare January 7, 2021 16:23
@lidel lidel added the need/review Needs a review label Jan 7, 2021
@lidel lidel force-pushed the fix/broken-cidv0-ux branch from 2cb958d to f31d9b1 Compare January 7, 2021 19:09
@lidel lidel marked this pull request as ready for review January 7, 2021 19:48
@aschmahmann aschmahmann merged commit 651e3cc into master Jan 14, 2021
@aschmahmann aschmahmann deleted the fix/broken-cidv0-ux branch January 14, 2021 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

need/review Needs a review topic/cidv1b32 Topic cidv1b32

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve error when broken CIDv0 is used

2 participants