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

Add Windows signed binaries support #32

Open
junosuarez opened this issue May 12, 2015 · 5 comments
Open

Add Windows signed binaries support #32

junosuarez opened this issue May 12, 2015 · 5 comments
Labels
build-target:windows Bundling an Electron app specifically for Windows enhancement Feature request help wanted Needs a contributor from the community

Comments

@junosuarez
Copy link
Contributor

see #31

It would be awesome to be able to output signed binaries.

I'm not positive if it's possible to sign executables after compilation with the current rcedit strategy we use. TODO: investigate this.

I'm actually pretty unfamiliar with the Visual Studio code signing tools overall. If anyone else knows what they're doing, feel free to pitch in! Otherwise, I'll try to investigate when I get time.

@junosuarez junosuarez added the help wanted Needs a contributor from the community label May 12, 2015
@Rameshv
Copy link
Contributor

Rameshv commented Jun 10, 2015

I have too many issues with https://github.com/atom/grunt-electron-installer for creating installers and code signing. I can't get this even compile in my local/build servers.

So i went ahead and implemented my own app installer creation (using inno setup https://github.com/jrsoftware/issrc instead the atom squirrel) and code signing with signtool.. it works perfect... its currently a bit messy.

Its a set of separate grunt tasks now. If you are looking to go this way, let me know.. i ll integrate with electron-package (along with this #30) and make a pull request soon

@junosuarez
Copy link
Contributor Author

@Rameshv do you have this work in a repo somewhere? We'll be happy to work with you to review it and integrate it with electron-packager as cleanly as possible. We're currently not using grunt, and I'd like to avoid introducing large new dependencies like that if we don't need to.

@malept malept added enhancement Feature request build-target:windows Bundling an Electron app specifically for Windows labels Oct 23, 2015
@coclav
Copy link

coclav commented Nov 23, 2015

I am using electron-boilerplate but I trust the logic would be the same here. I sign my app twice. First the "app.exe" and then the "app_installer.exe"

var sign = childProcess.spawn(resourcesDir.path("signtool"),
    [
        'sign',
        '/f',
        resourcesDir.path("MyCertificate.p12"),
        '/p',
        'APassword',
        '/t',
        'http://timestamp.verisign.com/scripts/timstamp.dll',
        readyAppDir.path(manifest.productName + '.exe')
    ],
    {
            stdio: 'inherit'
    });

  sign.on('error', function (err) {
      if (err) {
          throw "sign error " + err;
      } else {
          throw err;
      }
  });
  sign.on('close', function () {
      gulpUtil.log('Sign done!');
      deferred.resolve();
  });

And then this is the "packaged" into the installer, that I will sign the same way.

@malept malept changed the title Add --sign to Windows Add Windows support for --sign Mar 8, 2016
@malept malept changed the title Add Windows support for --sign Add Windows signed binaries support Aug 21, 2016
@totallygideon
Copy link

totallygideon commented Jul 3, 2020

According to the Electron documentation, some tools already offer this.

Specifically:

Have you tried any of those?

@malept
Copy link
Member

malept commented Jul 3, 2020

Those are all for signing installers, not the Electron binary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build-target:windows Bundling an Electron app specifically for Windows enhancement Feature request help wanted Needs a contributor from the community
Projects
None yet
Development

No branches or pull requests

5 participants