Skip to content

Windows installer signing#27

Merged
btoews merged 13 commits intomasterfrom
windows-installer-signing
Oct 2, 2018
Merged

Windows installer signing#27
btoews merged 13 commits intomasterfrom
windows-installer-signing

Conversation

@btoews
Copy link
Contributor

@btoews btoews commented Oct 1, 2018

No description provided.

@jameswhite
Copy link

Is it safe to assume that the password on key.pem was randomly generated with generate-csr.sh and isn't stored in this repo, and that the pfx_password is only used for the export password in make-pfx.sh?

@btoews
Copy link
Contributor Author

btoews commented Oct 2, 2018

The private key is encrypted in two places here:

In /windows-installer/key.pem it is encrypted with the key generated in /windows-installer/genreate-csr.sh.

In /windows-installer/codesign.pfx it is encrypted with the password generated by /windows-installer/make-pfx.sh. This password is encrypted with our AppVeyor account's public key and included in /.appveyor.yml. AppVeyor decrypts this key and injects it into the CI job where the private key is decrypted and used for signing releases.

@btoews
Copy link
Contributor Author

btoews commented Oct 2, 2018

We were talking past each other. The missing link was that the pfx_password in /.appveyor.yml is encrypted with a public key associated with our AppVeyor account. This value is decrypted by AppVeyor and injected into the CI build at run time.

@btoews btoews merged commit e0db87f into master Oct 2, 2018
@btoews btoews deleted the windows-installer-signing branch October 2, 2018 17:56
@btoews btoews mentioned this pull request Oct 2, 2018
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.

3 participants