Skip to content

Commit

Permalink
src,win: allow 403 errors for arm64 node.lib
Browse files Browse the repository at this point in the history
The servers hosting the header packages for Electron return 403
instead of 404 for the constructed URL for arm64 node.lib for
older releases that do not support arm64.

PR-URL: #1934
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: João Reis <[email protected]>
  • Loading branch information
richardlau authored and rvagg committed Nov 18, 2019
1 parent ad0d182 commit 7cf5079
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/install.js
Original file line number Diff line number Diff line change
Expand Up @@ -323,7 +323,7 @@ function install (fs, gyp, argv, callback) {

req.on('error', done)
req.on('response', function (res) {
if (res.statusCode === 404) {
if (res.statusCode === 403 || res.statusCode === 404) {
if (arch === 'arm64') {
// Arm64 is a newer platform on Windows and not all node distributions provide it.
log.verbose(`${name} was not found in ${libUrl}`)
Expand Down

0 comments on commit 7cf5079

Please sign in to comment.