Skip to content
This repository has been archived by the owner on May 26, 2022. It is now read-only.

remove dependency on go-multicodec #26

Merged
merged 4 commits into from
Mar 22, 2019
Merged

remove dependency on go-multicodec #26

merged 4 commits into from
Mar 22, 2019

Conversation

Stebalien
Copy link
Member

It's unmaintained and pulls in github.com/whyrusleeping/cbor. This patch explicitly re-implements the minimally needed features.

Also fixes #12.

It's unmaintained and pulls in github.com/whyrusleeping/cbor. This patch
explicitly re-implements the minimally needed features.

Also fixes #12.
@ghost ghost assigned Stebalien Mar 22, 2019
@ghost ghost added the status/in-progress In progress label Mar 22, 2019
@Stebalien Stebalien requested a review from Kubuxu March 22, 2019 01:11
headerPSKv1 = mc.Header(pathPSKv1)
pathPSKv1 = []byte("/key/swarm/psk/1.0.0/")
pathBase16 = "/base16/"
pathBase64 = "/base64/"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Multicodec also supported bin. IDK how widely it was deployed but it shouldn't be a problem.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean, I can just support /bin/.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

License: MIT
Signed-off-by: Jakub Sztandera <kubuxu@protonmail.ch>
@ghost ghost assigned Kubuxu Mar 22, 2019
@Kubuxu Kubuxu merged commit 3136fce into master Mar 22, 2019
@ghost ghost removed the status/in-progress In progress label Mar 22, 2019
@Kubuxu Kubuxu deleted the feat/no-multicodec branch March 22, 2019 01:53
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.

Fix handling of Windows lineends
2 participants