centrifuge / pod

Go implementation of Centrifuge POD (Private Off-chain Data) node
https://docs.centrifuge.io/build/p2p-node/
MIT License
77 stars 35 forks source link

Replace secp256k1 by ed25519 #1534

Closed mikiquantum closed 2 years ago

mikiquantum commented 2 years ago

closes #1520

codecov[bot] commented 2 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (v1.4.0@9b00732). Click here to learn what that means. The diff coverage is n/a.

@@            Coverage Diff            @@
##             v1.4.0    #1534   +/-   ##
=========================================
  Coverage          ?   65.48%           
=========================================
  Files             ?      130           
  Lines             ?     9773           
  Branches          ?        0           
=========================================
  Hits              ?     6400           
  Misses            ?     2792           
  Partials          ?      581           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 9b00732...e2e4a2d. Read the comment docs.

mikiquantum commented 2 years ago

@NunoAlexandre that is a good question. The idea is that only new asset originators will use this new version, otherwise we would need to do a full migration of identities accross chains, and we didnt think it was worth it due to the current user load we have that use the go-node. So we do not need to support legacy documents.