Skip to content

ECDSA: Update for FIPS 186-5.#2863

Merged
briansmith merged 4 commits into
mainfrom
b/ecdsa-digest-scalar
Jun 30, 2026
Merged

ECDSA: Update for FIPS 186-5.#2863
briansmith merged 4 commits into
mainfrom
b/ecdsa-digest-scalar

Conversation

@briansmith

Copy link
Copy Markdown
Owner

Reorder steps and cite FIPS 186-5 instead of older documentation.

@briansmith briansmith self-assigned this Jun 27, 2026
@codecov

codecov Bot commented Jun 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.44%. Comparing base (fc6c02f) to head (426ab55).

Files with missing lines Patch % Lines
src/ec/suite_b/ecdsa/signing.rs 66.66% 1 Missing and 1 partial ⚠️
src/ec/suite_b/ecdsa/verification.rs 90.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2863      +/-   ##
==========================================
- Coverage   96.45%   96.44%   -0.02%     
==========================================
  Files         204      204              
  Lines       20409    20402       -7     
  Branches      521      521              
==========================================
- Hits        19685    19676       -9     
- Misses        601      602       +1     
- Partials      123      124       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@briansmith briansmith force-pushed the b/ecdsa-digest-scalar branch 2 times, most recently from 8c0e72f to 9c8b6e2 Compare June 29, 2026 19:11
Split the computation of the digest from the conversion to a scalar.
Simplify the internal `digest_scalar` API by removing its dependency
on `Digest`.

Add test coverage of P-256 with SHA-384 and P-384 with SHA-256.
Remove SHA-1, SHA-224, and SHA-512 test vectors since we don't
support these algorithms in ECDSA signatures.
@briansmith briansmith force-pushed the b/ecdsa-digest-scalar branch from 9c8b6e2 to 426ab55 Compare June 30, 2026 00:07
@briansmith briansmith merged commit 6e549bc into main Jun 30, 2026
173 of 175 checks passed
@briansmith briansmith deleted the b/ecdsa-digest-scalar branch June 30, 2026 22:09
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.

1 participant