fix: logical bug in install.js env var handling (#48671)
If either `npm_config_electron_use_remote_checksums` or `electron_use_remote_checksums` are set as environment variables, then force Electron to verify with remote checksums instead of embedded ones. Fixes #48594. Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Charles Kerr <charles@charleskerr.com>
This commit is contained in:
parent
441e317400
commit
60a8cde048
1 changed files with 1 additions and 1 deletions
|
|
@ -44,7 +44,7 @@ downloadArtifact({
|
||||||
artifactName: 'electron',
|
artifactName: 'electron',
|
||||||
force: process.env.force_no_cache === 'true',
|
force: process.env.force_no_cache === 'true',
|
||||||
cacheRoot: process.env.electron_config_cache,
|
cacheRoot: process.env.electron_config_cache,
|
||||||
checksums: process.env.electron_use_remote_checksums ?? process.env.npm_config_electron_use_remote_checksums ? undefined : require('./checksums.json'),
|
checksums: (process.env.electron_use_remote_checksums || process.env.npm_config_electron_use_remote_checksums) ? undefined : require('./checksums.json'),
|
||||||
platform,
|
platform,
|
||||||
arch
|
arch
|
||||||
}).then(extractFile).catch(err => {
|
}).then(extractFile).catch(err => {
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue