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: windows signature verification special chars #8409

Conversation

ckarich
Copy link
Contributor

@ckarich ckarich commented Aug 10, 2024

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
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 #8162

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
Copy link

changeset-bot bot commented Aug 10, 2024

🦋 Changeset detected

Latest commit: d04b0b5

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 Aug 10, 2024

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

Name Link
🔨 Latest commit d04b0b5
🔍 Latest deploy log https://app.netlify.com/sites/car-park-attendant-cleat-11576/deploys/66b7a4c76d1ab7000875940c
😎 Deploy Preview https://deploy-preview-8409--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 merged commit 5fae1cf into electron-userland:master Aug 11, 2024
13 checks passed
@ckarich ckarich deleted the fix-win-signature-verification-(again) branch August 26, 2024 12:07
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.

Certificates with special characters are not accepted anymore since electron-updater v1.6.9
2 participants