Skip to content
This repository has been archived by the owner on Jul 21, 2023. It is now read-only.

fix: use node crypto for ed25519 signing and verification #289

Merged
merged 5 commits into from
Dec 1, 2022
Merged

fix: use node crypto for ed25519 signing and verification #289

merged 5 commits into from
Dec 1, 2022

Commits on Dec 1, 2022

  1. fix: use node crypto for ed25519 signing and verification 

    In node, use the node crypto module for ed25519 signing/verification
now that it's in all LTS releases.

Browsers still use the pure-js `@noble/ed25519` implementation.

Before:

```
@libp2p/crypto x 484 ops/sec ±0.34% (90 runs sampled)
```

After:

```
@libp2p/crypto x 4,706 ops/sec ±0.81% (84 runs sampled)
```
    @achingbrain
    achingbrain committed Dec 1, 2022
    Configuration menu
    Copy the full SHA
    8d8aff6 View commit details
    Browse the repository at this point in the history

  2. Merge remote-tracking branch 'origin/master' into fix/use-node-crypto… 

    …-for-ed25119
    @achingbrain
    achingbrain committed Dec 1, 2022
    Configuration menu
    Copy the full SHA
    9f48d57 View commit details
    Browse the repository at this point in the history

  3. chore: pr comments 

    Co-authored-by: Marin Petrunić <[email protected]>
    @achingbrain @mpetrunic
    achingbrain and mpetrunic authored Dec 1, 2022
    Configuration menu
    Copy the full SHA
    dd761a7 View commit details
    Browse the repository at this point in the history

  4. chore: avoid array copy

    @achingbrain
    achingbrain committed Dec 1, 2022
    Configuration menu
    Copy the full SHA
    56e24b8 View commit details
    Browse the repository at this point in the history

  5. chore: replace all .slice with .subarray

    @achingbrain
    achingbrain committed Dec 1, 2022
    Configuration menu
    Copy the full SHA
    a5e6dc5 View commit details
    Browse the repository at this point in the history