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

fix: auto-update powershell script requires reset of PSModulePath #8051

Merged
merged 13 commits into from
Feb 17, 2024

Conversation

mmaietta
Copy link
Collaborator

Fixes: #7127

Also implements additional unit tests, and signature verification on windows-latest build machines

Copy link

changeset-bot bot commented Feb 14, 2024

🦋 Changeset detected

Latest commit: 7bcf15a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
electron-updater Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

netlify bot commented Feb 14, 2024

Deploy Preview for car-park-attendant-cleat-11576 ready!

Name Link
🔨 Latest commit 7bcf15a
🔍 Latest deploy log https://app.netlify.com/sites/car-park-attendant-cleat-11576/deploys/65d10968e20ade000860ad47
😎 Deploy Preview https://deploy-preview-8051--car-park-attendant-cleat-11576.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mmaietta mmaietta marked this pull request as ready for review February 17, 2024 19:40
@mmaietta mmaietta merged commit 48603ba into master Feb 17, 2024
15 checks passed
@mmaietta mmaietta deleted the fix/windows-signature-verification branch February 17, 2024 19:41
@github-actions github-actions bot mentioned this pull request Feb 17, 2024
ckarich pushed a commit to ckarich/electron-builder that referenced this pull request Aug 10, 2024
by applying electron-userland#8051 and noted in electron-userland#8162 win signature
verification of signatures created using
certificates including special chars like german
umlauts got broken in electron-updater >= 1.6.9
issue at hand is that the windows set command does
not terminate when reading semicolon;
setting %PATH% is a good example for this
c.f. https://learn.microsoft.com/en-us/previous-versions/windows/it-pro/windows-xp/bb490998(v=technet.10)
so instead commands should be concatenated using &

Closes electron-userland#8162
mmaietta pushed a commit that referenced this pull request Aug 11, 2024
* fix: windows signature verification special chars

by applying #8051 and noted in #8162 win signature verification of signatures created using certificates including special chars like german umlauts got broken in electron-updater >= 1.6.9. The issue at hand is that the windows set command does not terminate when reading semicolon `;` and setting %PATH% is a good example for this
c.f. https://learn.microsoft.com/en-us/previous-versions/windows/it-pro/windows-xp/bb490998(v=technet.10) so instead commands should be concatenated using `&`

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

Successfully merging this pull request may close these issues.

Auto update failed while executing the powershell command
1 participant