Remove dependency on request.

We didn't really need it,
and one of its dependencies had a security issue.

Author: RubenVerborgh

lib/ldp-copy.js

@@ -4,7 +4,8 @@ const debug = require('./debug')
 const fs = require('fs')
 const mkdirp = require('fs-extra').mkdirp
 const path = require('path')
-const request = require('request')
+const http = require('http')
+const https = require('https')
 
 /**
  * Cleans up a file write stream (ends stream, deletes the file).
@@ -42,6 +43,7 @@ function copy (copyToPath, copyFromUri, callback) {
         debug.handlers('COPY -- Wrote data to: ' + copyToPath)
         callback()
       })
+    const request = /^https:/.test(copyFromUri) ? https : http
     request.get(copyFromUri)
       .on('error', function (err) {
         debug.handlers('COPY -- Error requesting source file: ' + err)
@@ -59,7 +61,7 @@ function copy (copyToPath, copyFromUri, callback) {
           error.statusCode = response.statusCode
           return callback(error)
         }
+        response.pipe(destinationStream)
       })
-      .pipe(destinationStream)
   })
 }