-
Notifications
You must be signed in to change notification settings - Fork 22
Async Crypto Endeavour (+benchmarks!) #19
Conversation
Replace lib multihashing with multihashing-async
Using string instead of buffer for proto file
There seems to be an issue with go interop on the verification step currently. Investigating currently. |
All good, go interop restored in the latest version. |
const suite = new Benchmark.Suite('secio') | ||
const ids = [] | ||
|
||
suite.add('createKey', function (d) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is d
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see the other PR or https://benchmarkjs.com/docs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean, can it have a better and intuitive name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just missing the Sauce Labs badge
cb(null, Buffer.concat([ | ||
data, | ||
digest | ||
])) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this could be one line, it is just a Buffer.concat
@diasdavid should be ready |
Removes all external crypto libraries, except for those used in
libp2p-crypto
andmultihashing
. This means it drops the usage of node-forge and relies only on webcrypto in the browser.Needs libp2p/js-libp2p-crypto#10
Stats
Size
1.8M
999K
606K
382K
Performance