From 25a8554be1550e00fba023fd56a3a6e181888e81 Mon Sep 17 00:00:00 2001 From: Alex Browne Date: Mon, 13 Aug 2018 14:17:05 -0700 Subject: fix(sol-compiler, sol-resolver): Bug where sol-resolver tried to read a directory (#961) fix(sol-compiler, sol-resolver): Fix bug where sol-resolver tried to read a directory --- packages/sol-resolver/CHANGELOG.json | 4 ++++ packages/sol-resolver/src/resolvers/npm_resolver.ts | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) (limited to 'packages/sol-resolver') diff --git a/packages/sol-resolver/CHANGELOG.json b/packages/sol-resolver/CHANGELOG.json index abce53d09..dd9cb3f5d 100644 --- a/packages/sol-resolver/CHANGELOG.json +++ b/packages/sol-resolver/CHANGELOG.json @@ -5,6 +5,10 @@ { "note": "Fix a bug where RelativeFSResolver would crash when trying to read a directory", "pr": 909 + }, + { + "note": "Fix a bug where NpmResolver would crash when trying to read a directory", + "pr": 961 } ] }, diff --git a/packages/sol-resolver/src/resolvers/npm_resolver.ts b/packages/sol-resolver/src/resolvers/npm_resolver.ts index 9f8617145..a2df0dcad 100644 --- a/packages/sol-resolver/src/resolvers/npm_resolver.ts +++ b/packages/sol-resolver/src/resolvers/npm_resolver.ts @@ -19,7 +19,7 @@ export class NPMResolver extends Resolver { const ROOT_PATH = '/'; while (currentPath !== ROOT_PATH) { const lookupPath = path.join(currentPath, 'node_modules', packageName, pathWithinPackage); - if (fs.existsSync(lookupPath)) { + if (fs.existsSync(lookupPath) && fs.lstatSync(lookupPath).isFile()) { const fileContent = fs.readFileSync(lookupPath).toString(); return { source: fileContent, -- cgit v1.2.3