Skip to content

Downgrade to curve25519-dalek v4.1.3#7079

Merged
samkim-crypto merged 1 commit intoanza-xyz:masterfrom
samkim-crypto:curve25519-downgrade
Jul 22, 2025
Merged

Downgrade to curve25519-dalek v4.1.3#7079
samkim-crypto merged 1 commit intoanza-xyz:masterfrom
samkim-crypto:curve25519-downgrade

Conversation

@samkim-crypto
Copy link
Copy Markdown

@samkim-crypto samkim-crypto commented Jul 22, 2025

Problem

The curve25519-dalek crate was upgraded to v4.2.0 in #6914, but the v4.2.0 was yanked in crates.io because the hash_to_curve function does not follow the specification in the standard.

Summary of Changes

Downgrade to v4.1.3.

Fixes #

@samkim-crypto samkim-crypto marked this pull request as ready for review July 22, 2025 13:20
@samkim-crypto samkim-crypto requested a review from yihau July 22, 2025 13:21
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.3%. Comparing base (5222e94) to head (68c52c7).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7079   +/-   ##
=======================================
  Coverage    83.2%    83.3%           
=======================================
  Files         854      854           
  Lines      374686   374610   -76     
=======================================
+ Hits       312083   312092    +9     
+ Misses      62603    62518   -85     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@samkim-crypto samkim-crypto merged commit d1e9a3e into anza-xyz:master Jul 22, 2025
52 checks passed
puhtaytow pushed a commit to puhtaytow/agave that referenced this pull request Jul 24, 2025
apfitzge added a commit to apfitzge/agave that referenced this pull request Jul 24, 2025
apfitzge added a commit to apfitzge/agave that referenced this pull request Jul 24, 2025
apfitzge added a commit that referenced this pull request Jul 24, 2025
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