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

Feature request: Force application signing #975

Closed
tarjei opened this issue Dec 5, 2016 · 8 comments · May be fixed by qcif/data-curator#563
Closed

Feature request: Force application signing #975

tarjei opened this issue Dec 5, 2016 · 8 comments · May be fixed by qcif/data-curator#563
Labels

Comments

@tarjei
Copy link

tarjei commented Dec 5, 2016

  • Version: 9.2.0
  • Target: windows/osx

It would be great to have a command line switch that fails the build if application signing fails. This could catch a lot of setup errors faster and better that other methods.

@develar
Copy link
Member

develar commented Dec 5, 2016

Not clear for what do you mean — if app cannot be signed — will be a build error.

@tarjei
Copy link
Author

tarjei commented Dec 5, 2016

Are you sure?

I had a script running that I expected should sign the app - but it didn't happen:

CSC_IDENTITY_AUTO_DISCOVERY=false
+ CSC_LINK=file://../certificates/comodo.p12
+ '[' '!' -f /some/path/bin/../comodo.p12.password ']'
++ cat /some/path/bin/../comodo.p12.password
+ CSC_KEY_PASSWORD=****
/some/path/bin/../node_modules/.bin/build -w
Skip app dependencies rebuild because platform is different
⚠️  "electron-squirrel-startup" dependency is not required for NSIS
Packaging for win32 x64 using electron 1.4.10 to dist/win-unpacked
Packaging NSIS installer for arch x64
Building NSIS installer

Here, I suspect the CSC_LINK is wrong or somehow not found in the environment variables of the app. I want a commandline option that says "if signing fails or is not enabled (by lack of env vars) then fail".

T

@develar
Copy link
Member

develar commented Dec 5, 2016

@tarjei It is already fixed, please try 10.5.0

@tarjei
Copy link
Author

tarjei commented Dec 5, 2016

How is it fixed?

I.e. I managed to create a signed build with 9.2.0 as well. What I want is a surefire way to ensure that signing happens.

@tarjei tarjei changed the title Feature request: Fail if signing failed Feature request: Force application signing Dec 5, 2016
@tarjei
Copy link
Author

tarjei commented Dec 5, 2016

I updated the title to better reflect what I'm looking for.

@develar
Copy link
Member

develar commented Dec 5, 2016

How is it fixed?

We check that file exists. Please try.

What I want is a surefire way to ensure that signing happens.

Do you mean that you want to enable some mode, in which app must be or signed, or build failed? To ensure, that even if you forget to define CSC_LINK, you will be aware because CI will be broken?

@develar
Copy link
Member

develar commented Dec 5, 2016

I think, you want not CLI switch, but build.forceCodeSigning to avoid repeat such CLI flag in all build (i.e. the same for macOS).

@tarjei
Copy link
Author

tarjei commented Dec 5, 2016

Yes!

develar added a commit to develar/electron-builder that referenced this issue Dec 11, 2016
Babel config moved to .babelrc because:

 * jest invalidate cache on config file modification
 * json5 can be used instead of ugly raw json

config option added as preparation to additional config file in yaml.

Closes electron-userland#975
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants