BUG: This line make paperboy return 403 forbidden in node for windows

FIX: Check not only "/", as too "\"
nicokruger · Aug 5, 2011 · 221692a · 221692a
1 parent f2b5d7f
commit 221692a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/paperboy.js b/lib/paperboy.js
@@ -10,7 +10,7 @@ exports.filepath = function (webroot, url) {
   // Append index.html if path ends with '/'
   fp = path.normalize(path.join(webroot, (url.match(/\/$/)=='/')  ? url+'index.html' : url));
   // Sanitize input, make sure people can't use .. to get above webroot
-  if (webroot[webroot.length - 1] !== '/') webroot += '/';
+  if (/\/|\\$/.test(webroot[webroot.length - 1])) webroot += '/';
   if (fp.substr(0, webroot.length) != webroot)
     return(['Permission Denied', null]);
   else