-
Notifications
You must be signed in to change notification settings - Fork 2k
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
fix: windows code signing #8084
Conversation
9862eca
to
03ad723
Compare
This reverts commit ce51002.
545f083
to
088dc55
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe just add npm run build:app &&
for consistency, can do it in future if you need
@jackkav about:
We should not add it in this case. The approach is:
The unpacked files need to get codesigned in an intermediate step, and then we build the Squirrel package from the codesigned |
todo
windows codesigning process after merging this PR
The approach is:
package:windows:unpacked
to bakewindows-unpacked
folder (which runsnpm run build:app
) with all common baked insomnia filespackage:windows:dist
and point it to the samewindows-unpacked
folder (using electron-builder's--dir
flag) for baking the temporary squirrel related files, signing with SSL.com's ci-images viacustomSign.js
hook, bake the installer package, and sign that as well from hook.debugging
found that Update.exe and Insomnia.exe (squirrel stub) are only available to codesign via hook
Solution
We use the container image for EV codesigner from a custom-hook during packaging.
Afterwards, both the Stub, Updater.exe and the installer are all code-signed in a single go:
(using upstream branch instead of fork to test, closed #8069)