module: allow long paths for require on Windows

#1801 introduced internal fs methods to speed up require. The methods do not call path._makeLong like their counterpart from the fs module. This brings back the old behaviour. Fixes: #1990 Fixes: #1980 Fixes: #1849 PR-URL: https://github.com/nodejs/io.js/pull/1991/files Reviewed-By: Bert Belder <[email protected]>
nodejs · Jun 16, 2015 · 671e64a · 671e64a · rvagg · Jun 22, 2015
1 parent 52a822d
commit 671e64a
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 3 deletions.
diff --git a/lib/module.js b/lib/module.js
@@ -67,7 +67,7 @@ function readPackage(requestPath) {
   }
 
   var jsonPath = path.resolve(requestPath, 'package.json');
-  var json = internalModuleReadFile(jsonPath);
+  var json = internalModuleReadFile(path._makeLong(jsonPath));
 
   if (json === undefined) {
     return false;
@@ -100,7 +100,7 @@ Module._realpathCache = {};
 
 // check if the file exists and is not a directory
 function tryFile(requestPath) {
-  const rc = internalModuleStat(requestPath);
+  const rc = internalModuleStat(path._makeLong(requestPath));
   return rc === 0 && toRealPath(requestPath);
 }
 
@@ -146,7 +146,7 @@ Module._findPath = function(request, paths) {
     var filename;
 
     if (!trailingSlash) {
-      const rc = internalModuleStat(basePath);
+      const rc = internalModuleStat(path._makeLong(basePath));
       if (rc === 0) {  // File.
         filename = toRealPath(basePath);
       } else if (rc === 1) {  // Directory.

diff --git a/test/parallel/test-require-long-path.js b/test/parallel/test-require-long-path.js
@@ -0,0 +1,17 @@
+'use strict';
+var common = require('../common');
+var fs = require('fs');
+var path = require('path');
+var assert = require('assert');
+
+// make a path that is more than 260 chars long.
+var fileNameLen = Math.max(261 - common.tmpDir.length - 1, 1);
+var fileName = path.join(common.tmpDir, new Array(fileNameLen + 1).join('x'));
+var fullPath = path.resolve(fileName);
+
+common.refreshTmpDir();
+fs.writeFileSync(fullPath, 'module.exports = 42;');
+
+assert.equal(require(fullPath), 42);
+
+fs.unlinkSync(fullPath);