fix: replace url-join with path.posix.join (#334)

webpack · Sep 21, 2018 · d75802b · d75802b
1 parent 0f9e2e7
commit d75802b
Show file tree

Hide file tree

Showing 5 changed files with 465 additions and 492 deletions.
diff --git a/lib/middleware.js b/lib/middleware.js
@@ -1,7 +1,7 @@
 'use strict';
 
+const path = require('path');
 const mime = require('mime');
-const urlJoin = require('url-join');
 const DevMiddlewareError = require('./DevMiddlewareError');
 const { getFilenameFromUrl, handleRangeHeaders, handleRequest, ready } = require('./util');
 
@@ -52,7 +52,7 @@ module.exports = function wrapper(context) {
                 throw new DevMiddlewareError('next');
               }
 
-              filename = urlJoin(filename, index);
+              filename = path.posix.join(filename, index);
               stat = context.fs.statSync(filename);
               if (!stat.isFile()) {
                 throw new DevMiddlewareError('next');

diff --git a/lib/util.js b/lib/util.js
@@ -4,7 +4,6 @@ const path = require('path');
 const { parse } = require('url');
 const querystring = require('querystring');
 const parseRange = require('range-parser');
-const urlJoin = require('url-join');
 
 const HASH_REGEXP = /[0-9a-f]{10,}/;
 
@@ -92,7 +91,7 @@ module.exports = {
     if (process.platform === 'win32') {
       // Path Handling for Microsoft Windows
       if (filename) {
-        uri = urlJoin((outputPath || ''), querystring.unescape(filename));
+        uri = path.posix.join((outputPath || ''), querystring.unescape(filename));
 
         if (!path.win32.isAbsolute(uri)) {
           uri = `/${uri}`;
@@ -104,7 +103,7 @@ module.exports = {
 
     // Path Handling for all other operating systems
     if (filename) {
-      uri = urlJoin((outputPath || ''), filename);
+      uri = path.posix.join((outputPath || ''), filename);
 
       if (!path.posix.isAbsolute(uri)) {
         uri = `/${uri}`;