Skip to content

Conversation

username-is-already-taken2
Copy link
Contributor

@username-is-already-taken2 username-is-already-taken2 commented Aug 4, 2025

Summary
This PR updates the Windows file signing process from DigiCert to Azure Trusted Signer.

Key Changes

  • Removed redundant tasks: Some tasks became unnecessary after switching to Azure and were removed to simplify the workflow.

  • Azure-specific tasks: Added new tasks to handle authentication required for jsign to work with Azure.

    • Note: jsign was upgraded on the runner to version 7.1, which supports trusted signing.
      Task conditionals update: During development, some task inputs changed to booleans, breaking existing conditionals. These have been updated accordingly.
  • Goreleaser upgrade attempt:

    • Attempted to upgrade goreleaser, which worked for most targets.

    • However, the resulting RPM signatures were invalid, so we’ve remained pinned to version 2.5.1.

    • Comments were added in the code to clarify this limitation.

Workflow Improvements

  • New inputs were added to the workflow_dispatch to aid with troubleshooting:

    • use-test-cert: Signs the binary using our test certificate (not publicly trusted).

    • goreleaser-snapshot: Enables snapshot mode in goreleaser, bypassing tag checks so you can build from any commit.

    • upload-artifacts: Uploads binaries to the workflow run—useful for inspecting outputs when not publishing.

Signed-off-by: Gary Bright <gary@mondoo.com>
…for validation when not publishing

Signed-off-by: Gary Bright <gary@mondoo.com>
Signed-off-by: Gary Bright <gary@mondoo.com>
…apshot

Signed-off-by: Gary Bright <gary@mondoo.com>
Signed-off-by: Gary Bright <gary@mondoo.com>
Signed-off-by: Gary Bright <gary@mondoo.com>
@username-is-already-taken2 username-is-already-taken2 changed the title Switched Digicert Signing for Azure Trusted Signing Migrate Code Signing from DigiCert to Azure Trusted Signer Aug 5, 2025
@username-is-already-taken2 username-is-already-taken2 marked this pull request as ready for review August 5, 2025 14:28
Copy link

@sibuthomasmathew sibuthomasmathew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@username-is-already-taken2 username-is-already-taken2 merged commit 16c5640 into main Aug 6, 2025
9 checks passed
@username-is-already-taken2 username-is-already-taken2 deleted the gary/change-windows-signing branch August 6, 2025 11:36
@github-actions github-actions bot locked and limited conversation to collaborators Aug 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants