From bb77100672292481c348437b022d10d64d50a096 Mon Sep 17 00:00:00 2001 From: ekatek Date: Mon, 16 Jun 2014 21:03:07 -0700 Subject: [PATCH] DO NOT remove the : character when sanitizing paths. That breaks NPM for packages --- tools/builder.js | 2 +- tools/package-source.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/builder.js b/tools/builder.js index 62ed12db3a..f845ca61b3 100644 --- a/tools/builder.js +++ b/tools/builder.js @@ -102,7 +102,7 @@ _.extend(Builder.prototype, { // Basic sanitization if (part.match(/^\.+$/)) throw new Error("Path contains forbidden segment '" + part + "'"); - part = part.replace(/[^a-zA-Z0-9._-]/g, ''); + part = part.replace(/[^a-zA-Z0-9._\:-]/g, ''); // If at last component, pull extension (if any) off of part var ext = ''; diff --git a/tools/package-source.js b/tools/package-source.js index 63b6bf5a83..c26668de3b 100644 --- a/tools/package-source.js +++ b/tools/package-source.js @@ -15,7 +15,7 @@ var release = require('./release.js'); // XXX: This is a medium-term hack, to avoid having the user set a package name // & test-name in package.describe. We will change this in the new control file // world in some way. -var AUTO_TEST_POSTFIX = "-localtest"; +var AUTO_TEST_POSTFIX = ":test"; var isTestName = function (name) { var nameEnd = name.substr(name.length - AUTO_TEST_POSTFIX.length); return nameEnd === AUTO_TEST_POSTFIX;