Skip to content

Commit acc1455

Browse files
committed
module: fix require('.')
1 parent 2b63bcd commit acc1455

File tree

5 files changed

+10
-1
lines changed

5 files changed

+10
-1
lines changed

lib/module.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ Module._resolveLookupPaths = function(request, parent) {
205205
}
206206

207207
var start = request.substring(0, 2);
208-
if (start !== './' && start !== '..') {
208+
if (start !== '.' && start !== './' && start !== '..') {
209209
var paths = modulePaths;
210210
if (parent) {
211211
if (!parent.paths) parent.paths = [];
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
module.exports = require('./');
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
module.exports = require('.');
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
exports.value = 42;

test/parallel/test-require-extensions-main.js

+6
Original file line numberDiff line numberDiff line change
@@ -2,3 +2,9 @@ var common = require('../common');
22
var assert = require('assert');
33

44
require(common.fixturesDir + '/require-bin/bin/req.js');
5+
6+
var a = require(common.fixturesDir + '/module-require/relative/dot.js');
7+
var b = require(common.fixturesDir + '/module-require/relative/dot-slash.js');
8+
9+
assert.equal(a.value, 42);
10+
assert.equal(a, b, 'require(".") should resolve like require("./")');

0 commit comments

Comments
 (0)