Skip to content
This repository has been archived by the owner on Apr 16, 2020. It is now read-only.

Add notes for CIDv1 Base32 discussion #39

Closed
wants to merge 1 commit into from

Conversation

lidel
Copy link
Contributor

@lidel lidel commented Jul 10, 2018

Copied rough notes from hackpad for reference.
@lgierth @alanshaw @olizilla @diasdavid @kyledrake @kevina @whyrusleeping @magik6k @Kubuxu

PREVIEW

These notes are already outdated and published here only for the record of our progress.
We will continue CID discussion tomorrow as a part of unconf.

Look for a session proposal with CID in its name.

@kevina
Copy link
Contributor

kevina commented Jul 11, 2018

Additional Notes For Uncorf session:

@diasdavid seams okay with allowing a multibase prefix on CidV0. At least on what @lgierth told me from his brief conversion.

@whyrusleeping wants to first output base32 by default. Either using the version 0 prefix or just taking on the multibase prefix. The reason for this is because it works with the existing code. @kevina thinks that the javascript code should just be modified. He also considers the fact that CidV0 can not take a multibase prefix a break in the abstraction.

@kevina want's to emphases that the CidV1 migration can come later. For now lets just get everything in base 32, including CidV0! A lot of people want this!

@kevina
Copy link
Contributor

kevina commented Jul 11, 2018

@kevina we should also decide what multibase version of base32 to use. @kevina recommends base32 or b using lowercase letters. See https://github.com/multiformats/multibase for a list of multibase formats.

@lidel
Copy link
Contributor Author

lidel commented Jul 11, 2018

Closing this, as notes are superseded by ones from unconf session: https://hackmd.io/PZCWvHfWR0-XTufiDrRGsg?view
@kyledrake became the Captain and a key point of contact for CIDv1 endeavor.
We will probably continue the discussion in ipfs/kubo#4143

img_20180711_133252

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants