Changeset

7231:c018a44b822a

Merge 0.9->0.10
author Kim Alvefur <zash@zash.se>
date Thu, 03 Mar 2016 16:05:34 +0100
parents 7227:c6f9d694d778 (current diff) 7230:20246b139607 (diff)
children 7232:db0dc08929e7 7233:71ca252d9f69
files plugins/mod_http_files.lua
diffstat 1 files changed, 8 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/plugins/mod_http_files.lua	Wed Mar 02 16:43:42 2016 +0100
+++ b/plugins/mod_http_files.lua	Thu Mar 03 16:05:34 2016 +0100
@@ -56,6 +56,7 @@
 
 local urldecode = require "util.http".urldecode;
 function sanitize_path(path)
+	if not path then return end
 	local out = {};
 
 	local c = 0;
@@ -74,6 +75,9 @@
 			out[c] = component;
 		end
 	end
+	if path:sub(-1,-1) == "/" then
+		out[c+1] = "";
+	end
 	return "/"..table.concat(out, "/");
 end
 
@@ -88,12 +92,13 @@
 	local directory_index = opts.directory_index;
 	local function serve_file(event, path)
 		local request, response = event.request, event.response;
-		path = sanitize_path(path);
-		if not path then
+		local sanitized_path = sanitize_path(path);
+		if path and not sanitized_path then
 			return 400;
 		end
+		path = sanitized_path;
 		local orig_path = sanitize_path(request.path);
-		local full_path = base_path .. (path and "/"..path or ""):gsub("/", path_sep);
+		local full_path = base_path .. (path or ""):gsub("/", path_sep);
 		local attr = stat(full_path:match("^.*[^\\/]")); -- Strip trailing path separator because Windows
 		if not attr then
 			return 404;