Skip to content

build: complete test-signing lifecycle in CI - #3

Merged
FreddyJD merged 2 commits into
mainfrom
build/verify-test-signature
Sep 14, 2026
Merged

FreddyJD merged 2 commits into
mainfrom
build/verify-test-signature

Conversation

@FreddyJD

Copy link
Copy Markdown
Contributor

Summary

  • generate a per-build non-exportable RSA test-signing key
  • embed and verify the driver's SHA-256 Authenticode signature with SignTool
  • temporarily trust only the public certificate during CI verification
  • remove runner trust entries and explicitly delete private-key material
  • fail an always-run cleanup guard if signing material remains
  • document the split between CI signing and Roxy's UAC-gated target-machine trust installation

Validation

Note

SignTool /kp requires a Microsoft-rooted production driver-signing chain and correctly rejects a self-signed research certificate. This pipeline uses /pa to verify the embedded SHA-256 Authenticode signature and separately checks that the embedded signer thumbprint equals the generated certificate. Loading remains limited to Windows test-signing mode; production loading requires Microsoft attestation or WHQL signing.

FreddyJD and others added 2 commits September 14, 2026 06:31
Create a non-exportable per-build signing key, require SignTool kernel-policy verification in CI, and remove temporary machine trust and private key material before artifact upload.

Co-authored-by: Roxy <299891354+roxy-commits@users.noreply.github.com>
Use SignTool's Authenticode policy for self-signed research drivers and add an always-run guard that rejects leftover test certificates or private keys.

Co-authored-by: Roxy <299891354+roxy-commits@users.noreply.github.com>
@FreddyJD
FreddyJD merged commit 62126a8 into main Sep 14, 2026
4 checks passed
@FreddyJD
FreddyJD deleted the build/verify-test-signature branch September 14, 2026 10:44
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