add TODOs

Author: benmccann

lib/install.js

@@ -56,11 +56,13 @@ function place_binary(uri, targetDir, opts, callback) {
                     process.env.npm_config_proxy;
   let agent;
   if (proxyUrl) {
+    // TODO: replace with undici's ProxyAgent
     const { HttpsProxyAgent } = require('https-proxy-agent');
     agent = new HttpsProxyAgent(proxyUrl);
     log.http('download', 'proxy agent configured using: "%s"', proxyUrl);
   }
 
+  // TODO: use node's built-in fetch
   fetch(sanitized, { agent })
     .then((res) => {
       if (!res.ok) {

test/proxy-bcrypt.test.js

@@ -44,6 +44,7 @@ test('setup proxy server', (t) => {
   proxy.startServer({ port: proxyPort });
   process.env.https_proxy = process.env.http_proxy = proxyServer;
 
+  // TODO: replace with undici's ProxyAgent
   options.agent = new HttpsProxyAgent(proxyServer);
 
   // make sure the download directory deleted then create an empty one
@@ -65,6 +66,7 @@ test('verify node fetch with a proxy successfully downloads bcrypt pre-built', (
   const url = 'https://github.com/kelektiv/node.bcrypt.js/releases/download/v5.0.1/bcrypt_lib-v5.0.1-napi-v3-linux-x64-glibc.tar.gz';
 
   async function getBcrypt() {
+    // TODO: use node's built-in fetch
     const res = await fetch(url, options);
     if (res.status !== 200) {
       throw new Error(`fetch got error ${res.status}`);