migrate to new cbor lib

[whyrusleeping]

We should use https://github.com/polydawn/refmt, it was designed with a lot of our usecases in mind

[magik6k]

This lib could be also used for serialization of dagNodes (currently ipld modules are hard-coded with MarshalJson)

cc @Stebalien

[Stebalien]

Missing pieces:

• CBOR Tags: cbor tags? polydawn/refmt#7
• UTF-8 Strings: UTF-8 versus bytes polydawn/refmt#8 (we can write a validating adapter).
• Big Numbers?

[Stebalien]

Note: some quick benchmarking shows that the current library is slightly faster (especially when decoding). However, that may change when decoding/encoding directly to a datastructure rather than an object.

[phritz]

Checking on status here. This is still WIP?

[Stebalien]

Yes but it's getting close. I need to review and merge some CID changes (so we'll always just use cid.Cid instead of *cid.Cid). Then, we'll need to update the refmt version here and we should be good to go.

[dignifiedquire]

That is great news :) Cheers Friedel

…

On 10. Sep 2018, 18:26 -0700, Steven Allen ***@***.***>, wrote: Yes but it's getting close. I need to review and merge some CID changes (so we'll always just use cid.Cid instead of *cid.Cid). Then, we'll need to update the refmt version here and we should be good to go. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

[Stebalien]

Done.