Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PICARD-2760: Timestamp the Windows binaries when codesigning #2321

Merged
merged 2 commits into from
Sep 21, 2023

Conversation

phw
Copy link
Member

@phw phw commented Sep 21, 2023

Summary

  • This is a…
    • Bug fix
    • Feature addition
    • Refactoring
    • Minor / simple change (like a typo)
    • Other
  • Describe this change in 1-2 sentences:

Problem

Enable timestamping when code signing for Windows. Signed binaries with a timestamp will remain valid even after the certificates have expired as long as they get signed before expiration of the certificates.

For details see:

Solution

Use the timestamp server provided by ssl.com with SignTool

This ensures the final binaries are still accepted by the OS even after
the certificates have expired.
@phw phw force-pushed the PICARD-2760-windows-codesign-timestamping branch from 7822b4f to 1e47319 Compare September 21, 2023 07:35
This tool is more flexible then the than the Powershell commandlet and
unifies code signing between .exe and .appx packages.
@phw phw force-pushed the PICARD-2760-windows-codesign-timestamping branch from 1e47319 to 7556986 Compare September 21, 2023 07:48
@phw phw marked this pull request as ready for review September 21, 2023 08:54
@phw
Copy link
Member Author

phw commented Sep 21, 2023

Tested with the signed binaries on Windows 10 and 11. The binaries now show the timestamp in the file properties:

grafik

@phw phw merged commit 05d605d into metabrainz:master Sep 21, 2023
69 checks passed
@phw phw deleted the PICARD-2760-windows-codesign-timestamping branch September 21, 2023 09:02
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