mod_dialback: Correctly check if a connection was destroyed (thanks iron)
[prosody.git] / net / http / server.lua
index 00571bce6956b38a86487acf5f34963bd8b8bdb8..69908e4e3c4ab8cfa795d1f86f2869b9ef1032cc 100644 (file)
@@ -12,15 +12,13 @@ local xpcall = xpcall;
 local debug = debug;
 local tostring = tostring;
 local codes = require "net.http.codes";
-local _G = _G;
-local legacy_httpserver = require "net.httpserver";
 
 local _M = {};
 
 local sessions = {};
-local handlers = {};
-
 local listener = {};
+local hosts = {};
+local default_host;
 
 local function is_wildcard_event(event)
        return event:sub(-2, -1) == "/*";
@@ -34,54 +32,63 @@ setmetatable(events._handlers, {
        __index = function (handlers, curr_event)
                if is_wildcard_event(curr_event) then return; end -- Wildcard events cannot be fired
                -- Find all handlers that could match this event, sort them
-               -- and then put the array into handlers[event]
+               -- and then put the array into handlers[curr_event] (and return it)
                local matching_handlers_set = {};
                local handlers_array = {};
                for event, handlers_set in pairs(event_map) do
                        if event == curr_event or
                        is_wildcard_event(event) and is_wildcard_match(event, curr_event) then
                                for handler, priority in pairs(handlers_set) do
-                                       matching_handlers_set[handler] = { (select(2, event:gsub("/", "%1"))), priority };
+                                       matching_handlers_set[handler] = { (select(2, event:gsub("/", "%1"))), is_wildcard_event(event) and 0 or 1, priority };
                                        table.insert(handlers_array, handler);
                                end
                        end
                end
-               if #handlers_array == 0 then return; end
-               table.sort(handlers_array, function(b, a)
-                       local a_score, b_score = matching_handlers_set[a], matching_handlers_set[b];
-                       for i = 1, #a_score do
-                               if a ~= b then -- If equal, compare next score value
-                                       return a_score[i] < b_score[i];
+               if #handlers_array > 0 then
+                       table.sort(handlers_array, function(b, a)
+                               local a_score, b_score = matching_handlers_set[a], matching_handlers_set[b];
+                               for i = 1, #a_score do
+                                       if a_score[i] ~= b_score[i] then -- If equal, compare next score value
+                                               return a_score[i] < b_score[i];
+                                       end
                                end
-                       end
-                       return false;
-               end);
-               handlers[curr_event] = handlers_array;
+                               return false;
+                       end);
+               else
+                       handlers_array = false;
+               end
+               rawset(handlers, curr_event, handlers_array);
                return handlers_array;
        end;
        __newindex = function (handlers, curr_event, handlers_array)
                if handlers_array == nil
                and is_wildcard_event(curr_event) then
-                       -- Invalidate all matching
+                       -- Invalidate the indexes of all matching events
                        for event in pairs(handlers) do
                                if is_wildcard_match(curr_event, event) then
                                        handlers[event] = nil;
                                end
                        end
                end
+               rawset(handlers, curr_event, handlers_array);
        end;
 });
 
 local handle_request;
 local _1, _2, _3;
 local function _handle_request() return handle_request(_1, _2, _3); end
-local function _traceback_handler(err) log("error", "Traceback[http]: %s: %s", tostring(err), debug.traceback()); end
+
+local last_err;
+local function _traceback_handler(err) last_err = err; log("error", "Traceback[http]: %s: %s", tostring(err), debug.traceback()); end
+events.add_handler("http-error", function (error)
+       return "Error processing request: "..codes[error.code]..". Check your error log for more information.";
+end, -1);
 
 function listener.onconnect(conn)
        local secure = conn:ssl() and true or nil;
        local pending = {};
        local waiting = false;
-       local function process_next(last_response)
+       local function process_next()
                --if waiting then log("debug", "can't process_next, waiting"); return; end
                if sessions[conn] and #pending > 0 then
                        local request = t_remove(pending);
@@ -90,7 +97,7 @@ function listener.onconnect(conn)
                        --handle_request(conn, request, process_next);
                        _1, _2, _3 = conn, request, process_next;
                        if not xpcall(_handle_request, _traceback_handler) then
-                               conn:write("HTTP/1.0 503 Internal Server Error\r\n\r\nAn error occured during the processing of this request.");
+                               conn:write("HTTP/1.0 500 Internal Server Error\r\n\r\n"..events.fire_event("http-error", { code = 500, private_message = last_err }));
                                conn:close();
                        end
                else
@@ -117,6 +124,11 @@ function listener.onconnect(conn)
 end
 
 function listener.ondisconnect(conn)
+       local open_response = conn._http_open_response;
+       if open_response and open_response.on_destroy then
+               open_response.finished = true;
+               open_response:on_destroy();
+       end
        sessions[conn] = nil;
 end
 
@@ -154,50 +166,67 @@ function handle_request(conn, request, finish_cb)
                send = _M.send_response;
                finish_cb = finish_cb;
        };
+       conn._http_open_response = response;
 
-       if not request.headers.host then
-               response.status_code = 400;
-               response.headers.content_type = "text/html";
-               response:send("<html><head>400 Bad Request</head><body>400 Bad Request: No Host header.</body></html>");
-       else
-               local host = request.headers.host;
-               if host then
-                       host = host:match("[^:]*"):lower();
-                       local event = request.method.." "..host..request.path:match("[^?]*");
-                       local payload = { request = request, response = response };
-                       --log("debug", "Firing event: %s", event);
-                       local result = events.fire_event(event, payload);
-                       if result ~= nil then
-                               if result ~= true then
-                                       local code, body = 200, "";
-                                       local result_type = type(result);
-                                       if result_type == "number" then
-                                               response.status_code = result;
-                                       elseif result_type == "string" then
-                                               body = result;
-                                       elseif result_type == "table" then
-                                               body = result.body;
-                                               result.body = nil;
-                                               for k, v in pairs(result) do
-                                                       response[k] = v;
-                                               end
-                                       end
-                                       response:send(body);
+       local host = (request.headers.host or ""):match("[^:]+");
+
+       -- Some sanity checking
+       local err_code, err;
+       if not request.path then
+               err_code, err = 400, "Invalid path";
+       elseif not hosts[host] then
+               if hosts[default_host] then
+                       host = default_host;
+               elseif host then
+                       err_code, err = 404, "Unknown host: "..host;
+               else
+                       err_code, err = 400, "Missing or invalid 'Host' header";
+               end
+       end
+       
+       if err then
+               response.status_code = err_code;
+               response:send(events.fire_event("http-error", { code = err_code, message = err }));
+               return;
+       end
+
+       local event = request.method.." "..host..request.path:match("[^?]*");
+       local payload = { request = request, response = response };
+       --log("debug", "Firing event: %s", event);
+       local result = events.fire_event(event, payload);
+       if result ~= nil then
+               if result ~= true then
+                       local body;
+                       local result_type = type(result);
+                       if result_type == "number" then
+                               response.status_code = result;
+                               if result >= 400 then
+                                       body = events.fire_event("http-error", { code = result });
+                               end
+                       elseif result_type == "string" then
+                               body = result;
+                       elseif result_type == "table" then
+                               for k, v in pairs(result) do
+                                       response[k] = v;
                                end
-                               return;
                        end
+                       response:send(body);
                end
-
-               -- if handler not called, return 404
-               response.status_code = 404;
-               response.headers.content_type = "text/html";
-               response:send("<html><head><title>404 Not Found</title></head><body>404 Not Found: No such page.</body></html>");
+               return;
        end
+
+       -- if handler not called, return 404
+       response.status_code = 404;
+       response:send(events.fire_event("http-error", { code = 404 }));
 end
 function _M.send_response(response, body)
+       if response.finished then return; end
+       response.finished = true;
+       response.conn._http_open_response = nil;
+       
        local status_line = "HTTP/"..response.request.httpversion.." "..(response.status or codes[response.status_code]);
        local headers = response.headers;
-       body = body or "";
+       body = body or response.body or "";
        headers.content_length = #body;
 
        local output = { status_line };
@@ -208,6 +237,10 @@ function _M.send_response(response, body)
        t_insert(output, body);
 
        response.conn:write(t_concat(output));
+       if response.on_destroy then
+               response:on_destroy();
+               response.on_destroy = nil;
+       end
        if headers.connection == "Keep-Alive" then
                response:finish_cb();
        else
@@ -224,7 +257,17 @@ end
 function _M.listen_on(port, interface, ssl)
        addserver(interface or "*", port, listener, "*a", ssl);
 end
+function _M.add_host(host)
+       hosts[host] = true;
+end
+function _M.remove_host(host)
+       hosts[host] = nil;
+end
+function _M.set_default_host(host)
+       default_host = host;
+end
 
 _M.listener = listener;
 _M.codes = codes;
+_M._events = events;
 return _M;