Skip to content

Bump electron-builder to ^26.0.0 to fix Azure signing with spaces in path#86

Merged
simonhamp merged 1 commit intoNativePHP:mainfrom
igorgudelj:fix/azure-signing-spaces-in-path
Mar 4, 2026
Merged

Bump electron-builder to ^26.0.0 to fix Azure signing with spaces in path#86
simonhamp merged 1 commit intoNativePHP:mainfrom
igorgudelj:fix/azure-signing-spaces-in-path

Conversation

@igorgudelj
Copy link
Contributor

@igorgudelj igorgudelj commented Feb 28, 2026

Summary

  • Azure Trusted Signing fails when the project path contains spaces (e.g. C:\Users\John Doe\project). The -Files parameter passed to Invoke-TrustedSigning is not quoted, so PowerShell splits the path at the space.
  • This was fixed upstream in electron-builder#8606 (commit a0e635c), released in app-builder-lib v26.0.0.
  • Bumps electron-builder from ^25.1.8 to ^26.0.0 to pick up the fix.

Reproduction

  1. Have a project path with a space (e.g. C:\Users\John Doe\Herd\myapp)
  2. Configure Azure Trusted Signing
  3. Run php artisan native:publish win
  4. Signing fails with SignTool Error: File not found: C:\Users\John

Test plan

  • Build/publish a NativePHP app on Windows from a path containing spaces with Azure Trusted Signing configured
  • Build/publish a NativePHP app from a path without spaces (no regression)
  • Build/publish on macOS/Linux (no regression)

…path

Azure Trusted Signing fails when the project path contains spaces
(e.g. C:\Users\John Doe\project). The -Files parameter passed to
Invoke-TrustedSigning was not quoted, causing PowerShell to split
the path at the space.

This was fixed upstream in electron-builder#8606 (commit a0e635c),
released in app-builder-lib v26.0.0.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gwleuverink
Copy link
Collaborator

Great! We were just discussing this yesterday. Thank you 🙏

@simonhamp simonhamp merged commit a028946 into NativePHP:main Mar 4, 2026
3 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants