Skip to content

Commit

Permalink
Merge pull request #149 from 2colours/remove-request-dependency
Browse files Browse the repository at this point in the history
Remove dependency on "request" package
  • Loading branch information
DeeDeeG authored Feb 14, 2025
2 parents 1ec7d89 + 2b31540 commit 6981ce7
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 18 deletions.
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@
"open": "7.3.0",
"plist": "3",
"read": "~1.0.7",
"request": "^2.88.2",
"season": "^6.0.2",
"second-mate": "https://github.com/pulsar-edit/second-mate.git#9686771",
"semver": "^7.3.4",
Expand Down
36 changes: 19 additions & 17 deletions script/download-node.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ const path = require('path');
const tar = require('tar');
const temp = require('temp');

const request = require('request');
const superagent = require('superagent');
const getInstallNodeVersion = require('./bundled-node-version')

temp.track();
Expand All @@ -23,14 +23,15 @@ const downloadFileToLocation = function(url, filename, callback) {
const stream = fs.createWriteStream(filename);
stream.on('end', callback);
stream.on('error', callback);
const requestStream = request.get(url)
requestStream.on('response', function(response) {
if (response.statusCode === 404) {
console.error('download not found:', url);
process.exit(1);
}
requestStream.pipe(stream);
});
superagent
.get(url)
.on('response', response => {
if (response.statusCode === 404) {
console.error('download not found:', url);
process.exit(1);
}
})
.pipe(stream);
};

const downloadTarballAndExtract = function(url, location, callback) {
Expand All @@ -42,14 +43,15 @@ const downloadTarballAndExtract = function(url, location, callback) {
callback.call(this, tempPath);
});
stream.on('error', callback);
const requestStream = request.get(url)
requestStream.on('response', function(response) {
if (response.statusCode === 404) {
console.error('download not found:', url);
process.exit(1);
}
requestStream.pipe(zlib.createGunzip()).pipe(stream);
});
superagent
.get(url)
.on('response', response => {
if (response.statusCode === 404) {
console.error('download not found:', url);
process.exit(1);
}
})
.pipe(zlib.createGunzip()).pipe(stream);
};

const copyNodeBinToLocation = function(callback, version, targetFilename, fromDirectory) {
Expand Down

0 comments on commit 6981ce7

Please sign in to comment.