EACCES when using quitAndInstall electron 3

• Version:
  • Builder: 20.28.4
  • Updater: 3.1.2

• Target: Windows

C:\Users\tiguerin\AppData\Roaming\BatchExplorer\__update__\BatchExplorer Setup 0.18.3-stable.99.exe EACCES
        at Process.ChildProcess._handle.onexit (internal/child_process.js:229:19)
        at onErrorNT (internal/child_process.js:406:16)

I believe the issue is that the spawn command doesn’t actually throw an error that is catchable here https://github.com/electron-userland/electron-builder/blob/master/packages/electron-updater/src/NsisUpdater.ts#L115

After testing locally I think the try catch needs to be changed to on("error" might have been a change to node 10 vs 8

This mock code for example works fine when using .on("error"

const { spawn } = require("child_process");

const installerPath = "C:\\Users\\tiguerin\\AppData\\Roaming\\BatchExplorer\\__update__\\BatchExplorer Setup 0.18.3-stable.99.exe";
const args = [];
const spawnOptions = {
    detached: true,
    stdio: "ignore",
};

const elevatePath = "C:\\Program Files\\BatchExplorer\\resources\\elevate.exe";

spawn(installerPath, args, spawnOptions).on("error", (e) => {
    // yes, such errors dispatched not as error event
    // https://github.com/electron-userland/electron-builder/issues/1129
    if ((e).code === "UNKNOWN" || (e).code === "EACCES") { // Node 8 sends errors: https://nodejs.org/dist/latest-v8.x/docs/api/errors.html#errors_common_system_errors
        console.log("Access denied or UNKNOWN error code on spawn, will be executed again using elevate")

        try {
            spawn(elevatePath, [installerPath].concat(args), spawnOptions).unref()
        } catch (e) {
            console.error("Elevate failed too", e);
        }
    } else {
        console.error("Error here", e);
    }
})

[dscalzi]

I’ve compiled @timotheeguerin’s solution to another repository. https://github.com/WesterosCraftCode/electron-updater-bin

As a temporary workaround, you can change your electron-updater dependency to electron-updater-bin which points to that repo. Change any imports/requirements from electron-updater to electron-updater-bin.

When the PR is merged and released you can switch back to electron-updater.

[Pagebakers]

Thanks @dscalzi That works

If you rename the package back to electron-updater you can just run

npm install electron-updater@https://github.com/WesterosCraftCode/electron-updater-bin

Without needing to change anything in the code.