perf: ⚡️ replace request with axios (#314)

As I described in #234 `request` is slow and deprecated. This change
replaces it with `axios` which is a better alternative. This also
removes the proxy configuration because it wasn't even working (no
`.get()` when setting the defaults) and `axios` supports `http_proxy` by
default (I tested)

Author: rodrigograca31

lib/config.js

@@ -11,9 +11,6 @@ exports.get = () => {
 
   let defaultConfig = JSON.parse(fs.readFileSync(DEFAULT));
   defaultConfig.cache = path.join(osHomedir(), '.tldr');
-  /*eslint-disable no-process-env */
-  defaultConfig.proxy = process.env.HTTP_PROXY || process.env.http_proxy;
-  /*eslint-enable no-process-env */
 
   let customConfig = {};
   try {

lib/remote.js

@@ -1,32 +1,27 @@
 'use strict';
 
 const unzip = require('node-unzip-2');
-const config  = require('./config');
-
-let request = require('request');
+const config = require('./config');
+const axios = require('axios');
 
 // Downloads the zip file from github and extracts it to folder
 exports.download = (path) => {
-  let url = config.get().repository;
+  const url = config.get().repository;
 
   // Creating the extractor
-  let extractor = unzip.Extract({ path });
-
-  // Setting the proxy if set by config
-  if (config.get().proxy) {
-    request = request.defaults({ proxy: config.proxy });
-  }
+  const extractor = unzip.Extract({ path });
 
-  // Creating the request and passing the extractor
-  let req = request.get({
+  let req = axios({
+    method: 'get',
     url: url,
+    responseType: 'stream',
     headers: { 'User-Agent' : 'tldr-node-client' }
+  }).then(function (response) {
+    response.data.pipe(extractor);
   });
 
-  req.pipe(extractor);
-
   return new Promise((resolve, reject) => {
-    req.on('error', (err) => {
+    req.catch((err) => {
       reject(err);
     });
     extractor.on('error', () => {