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

Not signed binary files in package for Windows #2543

Closed
gemiusz opened this issue Sep 28, 2018 · 9 comments · Fixed by #16946
Closed

Not signed binary files in package for Windows #2543

gemiusz opened this issue Sep 28, 2018 · 9 comments · Fixed by #16946
Assignees
Labels

Comments

@gemiusz
Copy link

gemiusz commented Sep 28, 2018

Is this a Feature or Bug?

Security improvement.

Current behavior:

Windows binary files aren't signed by your certificate - easier for malware to "be Cypress".
In some organisations only signed exe files can be allowed to use.

Desired behavior:

Signed binary files in package for Windows.

Steps to reproduce:

None :)

Versions

Cypress 3.1.0
Windows 10

@Saibamen
Copy link
Contributor

Any update?

@jennifer-shehane jennifer-shehane added the stage: proposal 💡 No work has been done of this issue label Feb 25, 2019
@jennifer-shehane jennifer-shehane added stage: ready for work The issue is reproducible and in scope and removed stage: proposal 💡 No work has been done of this issue labels Feb 25, 2019
@jennifer-shehane
Copy link
Member

No work has been done on this issue

@jennifer-shehane
Copy link
Member

Since this issue hasn't had activity in a while, we'll close the issue until we can confirm this is still happening. Please comment if there is new information to provide concerning the original issue and we'd be happy to reopen.

@jennifer-shehane jennifer-shehane removed the stage: ready for work The issue is reproducible and in scope label Feb 3, 2021
@minijus
Copy link
Contributor

minijus commented May 12, 2021

@jennifer-shehane would it be possible to reopen this issue?

Our organisation is exactly such as described:

In some organisations only signed exe files can be allowed to use.

Please let me know if there is any help needed with requirements gathering. The release process does not seem to be open sourced, so it will not be possible to contribute with actual change, I guess.

Edit:
It seems Mac binary was affected by similar issue #5791 and it was solved by #6013.
@bahmutov @brian-mann would it be possible to configure electron-builder to code sign Windows app as well? Would you be accepting PRs for this?

@jennifer-shehane
Copy link
Member

@minijus We would be open to accepting PRs for this if it's not too heavy of a lift by our team to finish. I remember the signing of the Mac binary was not a simple task, it ended up taking months, but maybe it will be easier since we already set that up. We would need the process automated in a similar way.

@jennifer-shehane jennifer-shehane added the stage: proposal 💡 No work has been done of this issue label May 13, 2021
@minijus
Copy link
Contributor

minijus commented May 14, 2021

Just did a quick verification of passing needed data to electron-builder as per Code Signing documentation.

It seems that setting WIN_CSC_LINK and WIN_CSC_KEY_PASSWORD is enough to produce signed binaries.
After running yarn binary-build in local workspace with env variables mentioned above:
image

In order to continue Cypress should acquire Code Signing certificate, add it to CI system and 🥳.

Update:
@jennifer-shehane could Cypress have internal discussion to buy code signing certificate (~400 $/yr)?

minijus added a commit to minijus/cypress that referenced this issue May 25, 2021
minijus added a commit to minijus/cypress that referenced this issue May 25, 2021
minijus added a commit to minijus/cypress that referenced this issue May 25, 2021
@jennifer-shehane
Copy link
Member

jennifer-shehane commented May 25, 2021

The error when attempting to run cypress open in this situation where orgs only allow signed exe to be used shows like below from #16670 (comment):

Cypress failed to start

...

Command failed with UNKNOWN
spawn UNKNOWN

119539765-462c0980-bd95-11eb-906c-c980ad71175e

@cypress-bot cypress-bot bot added stage: work in progress stage: needs review The PR code is done & tested, needs review and removed stage: needs review The PR code is done & tested, needs review stage: work in progress labels Jun 16, 2021
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jun 21, 2021

The code for this is done in cypress-io/cypress#16946, but has yet to be released.
We'll update this issue and reference the changelog when it's released.

@cypress-bot cypress-bot bot removed the stage: needs review The PR code is done & tested, needs review label Jun 21, 2021
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jun 23, 2021

Released in 7.6.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v7.6.0, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Jun 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants