Skip to content

docs: correct signing timing assumptions and class-D rationale - #408

Merged
lannbot merged 1 commit into
mainfrom
docs/signing-timing-policy
Sep 8, 2026
Merged

docs: correct signing timing assumptions and class-D rationale#408
lannbot merged 1 commit into
mainfrom
docs/signing-timing-policy

Conversation

@lannbot

@lannbot lannbot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Decision

Keep Ed25519 signing exported and ECDSA signing withheld, but state the basis accurately: ECDSA's exclusion is conservative provider policy pending signing-path assurance, not an inherent inability to implement constant-time ECDSA or a measured safety difference from Ed25519.

Replace the incorrect absence-of-secret-nonce and algorithm-level constant-time claims across the WIT docs, provider policy, and Rust comments. RFC 8032 derives a secret per-message nonce deterministically; determinism removes the fresh-randomness requirement, not leakage risk. Record compiler/runtime assumptions and the lack of an Ed25519 signing timing-lab probe. Correct the adjacent cfg-versus-dependency/DCE explanation.

The pinned P-256 scalar inversion uses a fixed public exponent; P-384 uses fixed-iteration divstep. Source inspection supports constant-time-designed arithmetic, not an end-to-end timing guarantee. No generated-code assurance is claimed.

Documentation/comments only. No executable code, WIT signatures, exports, dependencies, or conformance expectations change. CBC timing policy is a separate change.

Verification

  • just fmt-check
  • just validate-wit
  • cargo check -p polymorph-webcrypto-core -p polymorph-webcrypto-guest-provider
  • git diff --check
  • Independent documentation review; no blocking findings.

Automerge will be armed and CI watched.

Fixes #404

@lannbot
lannbot enabled auto-merge September 8, 2026 17:06
@lannbot
lannbot merged commit 988515e into main Sep 8, 2026
10 checks passed
@lannbot
lannbot deleted the docs/signing-timing-policy branch September 8, 2026 17:12
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.

Security docs: Ed25519 nonce and constant-time claims overstate the class-B rationale

2 participants