Skip to content

bump k256 and p256 to stable releases - #1486

Merged
arvidn merged 1 commit into
mainfrom
p256-k256-stable
Jul 13, 2026
Merged

arvidn merged 1 commit into
mainfrom
p256-k256-stable

Conversation

@arvidn

@arvidn arvidn commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

bump to from 0.14.0-rc -> 0.14.0

To no longer depend on release candidates


Note

Medium Risk
Touches ECDSA signing dependencies and the prehash signing path; behavior should match stable releases but warrants crypto/signing regression checks.

Overview
Moves workspace k256 and p256 from 0.14.0-rc to 0.14.0, with Cargo.lock refreshed for the stable RustCrypto stack (ecdsa, elliptic-curve, signature, rfc6979, and related crates).

In chia-secp, sign_prehashed for secp256k1 and secp256r1 no longer uses ? on sign_prehash_recoverable—the stable API returns the recoverable signature tuple directly, so the wrappers take .0 without unwrapping a Result.

Reviewed by Cursor Bugbot for commit 495cc15. Bugbot is set up for automated code reviews on this repo. Configure here.

@socket-security

Copy link
Copy Markdown

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

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedcargo/​k256@​0.14.0-rc.9 ⏵ 0.14.010010093100100
Updatedcargo/​p256@​0.14.0-rc.9 ⏵ 0.14.010010093100100

View full report

@coveralls-official

Copy link
Copy Markdown

Coverage Report for CI Build 29031828155

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage remained the same at 81.938%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 2 of 2 lines across 2 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 19671
Covered Lines: 16118
Line Coverage: 81.94%
Coverage Strength: 11507561.55 hits per line

💛 - Coveralls

@arvidn
arvidn requested a review from Rigidity July 10, 2026 06:24
@arvidn
arvidn merged commit 181b487 into main Jul 13, 2026
84 checks passed
@arvidn
arvidn deleted the p256-k256-stable branch July 13, 2026 13:40
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.

2 participants