Skip to content

Conversation

@Shaptic
Copy link
Contributor

@Shaptic Shaptic commented May 28, 2025

Having two methods is janky, sodium-native doesn't build on some systems, and @noble/curves is an audited, modern, high-performance library that achieves the same goals.

@Shaptic Shaptic requested a review from a team May 28, 2025 21:40
@socket-security
Copy link

socket-security bot commented May 28, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​noble/​hashes@​1.8.010010010085100
Added@​noble/​curves@​1.9.110010010089100

View full report

@github-actions
Copy link

github-actions bot commented May 28, 2025

Size Change: +101 kB (+3.03%)

Total Size: 3.42 MB

Filename Size Change
dist/stellar-base.js 2.52 MB +88.9 kB (+3.65%)
dist/stellar-base.min.js 893 kB +11.7 kB (+1.33%)

compressed-size-action

@Shaptic Shaptic merged commit 52222ca into protocol-23 Jun 12, 2025
7 checks passed
@Shaptic Shaptic deleted the migrate-crypto branch June 12, 2025 15:53
Shaptic added a commit that referenced this pull request Jun 18, 2025
* Migrate to @noble/curves instead of sodium+nacl
Shaptic added a commit that referenced this pull request Jun 25, 2025
* Add strkey support for liquidity pools and claimable balances (#799)
* Regenerate XDR with latest Protocol 23 version (#800)
* Migrate cryptographic libraries to use modern @noble/curves. (#802)
* Add support for encoding `Address` to/from new `StrKey`s (#801)
* Allow specifying mixed types for vectors in `nativeToScVal` (#803)
* Prepare v14.0.0-rc.1, Protocol 23 for release (#804)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants