azures04/crafatar
1
0
Fork 0
mirror of https://github.com/azures04/crafatar.git synced 2026-03-22 07:51:17 +01:00
Code Issues Packages Projects Releases Wiki Activity

get rid of path.resolve

Browse Source 
This doesn't handle '..' and other things that
path.resolve does, but if you're passing weird
things to a URL, your fault.
This commit is contained in:
jomo 2015-04-25 15:49:47 +02:00
parent 22ea5a7e27
commit 79ab296f1f
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/server.js
View File

@ -44,9 +44,8 @@ function request_id() {
// splits a URL path into an Array // splits a URL path into an Array
// the path is resolved and decoded // the path is resolved and decoded
function path_list(pathname) { function path_list(pathname) {
// remove trailing and double slashes + other junk // remove double and trailing slashes
pathname = pathname.replace(/\/\/+/g, "/").replace(/\/$/, "");
pathname = path.resolve(pathname);
var list = pathname.split("/"); var list = pathname.split("/");
list.shift(); list.shift();
for (var i = 0; i < list.length; i++) { for (var i = 0; i < list.length; i++) {