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

/n signtool param support #590

Closed
vpj opened this issue Jul 15, 2016 · 6 comments
Closed

/n signtool param support #590

vpj opened this issue Jul 15, 2016 · 6 comments

Comments

@vpj
Copy link

vpj commented Jul 15, 2016

electron-builder is using electron-winstaller-fixed npm module. This module links to electron-winstaller repo. So I assume it is a fork with some fixes. I couldn't find the actual repository of electron-winstaller-fixed.

I wanted to sign my code with a EV certificate on windows and it seemed easier to use signWithParams option of electron-winstaller. This option is not available under latest electron-winstaller-fixed and it no longer uses signtool even when building under windows. So I used electron-builder with electron-winstaller instead of electron-winstaller-fixed. And it worked without a problem.

I'm using electron-builder version 5.13.1 which depends on electron-winstaller-fixed version 3.0.0.

Is there a particular reason why electron-winstaller-fixed is used? Also how do you sign with a EV certificate (one with a USB token)?

@develar
Copy link
Member

develar commented Jul 19, 2016

We don't use signWithParams because on Linux and macOS we use another tool to sign.

EV certificate

Please provide your signWithParams and it will be supported without such param. Why EV requires signWithParams?

@vpj
Copy link
Author

vpj commented Jul 19, 2016

This is my signWithParams value.

/t http://timestamp.digicert.com /n "NAME"

I'm new to electron builder, and I couldn't find any other options in [Options)(https://github.com/electron-userland/electron-builder/wiki/Options) or Code Signing documents.

@develar
Copy link
Member

develar commented Jul 19, 2016

@vpj Just set certificateFile and certificatePassword (in the build.win). Or CSC_LINK

@develar develar closed this as completed Jul 19, 2016
@develar develar reopened this Jul 19, 2016
@develar
Copy link
Member

develar commented Jul 19, 2016

/n do you mean that you don't have p12 file, but instead you want to get cert from "The following command signs the file using a certificate whose private key information is protected by a hardware cryptography module. A computer store is specified for the certification authority (CA) store."?

@vpj
Copy link
Author

vpj commented Jul 19, 2016

Yes, I do not have a p12 file, and yes it comes with a usb device.

@develar develar changed the title Why not use electron-winstaller? /n signtool param support Jul 19, 2016
develar added a commit to develar/electron-builder that referenced this issue Aug 9, 2016
@develar develar closed this as completed Aug 9, 2016
@develar
Copy link
Member

develar commented Aug 10, 2016

certificateSubjectName

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants