Merge with 0.5

[prosody.git] / net / httpserver.lua

diff --git a/net/httpserver.lua b/net/httpserver.lua
index 77cc5b981ad5a5016767401f3013fa3b6ba5c0f6..56dfe04efdb031522bdd7c9ce7b3280de08af07c 100644 (file)
--- a/net/httpserver.lua
+++ b/net/httpserver.lua
@@ -37,6 +37,7 @@ local function send_response(request, response)
        -- Write status line
        local resp;
        if response.body then
+               local body = tostring(response.body);
                log("debug", "Sending response to %s", request.id);
                resp = { "HTTP/1.0 ", response.status or "200 OK", "\r\n"};
                local h = response.headers;
@@ -48,19 +49,19 @@ local function send_response(request, response)
                                t_insert(resp, "\r\n");
                        end
                end
-               if response.body and not (h and h["Content-Length"]) then
+               if not (h and h["Content-Length"]) then
                        t_insert(resp, "Content-Length: ");
-                       t_insert(resp, #response.body);
+                       t_insert(resp, #body);
                        t_insert(resp, "\r\n");
                end
                t_insert(resp, "\r\n");
                
-               if response.body and request.method ~= "HEAD" then
-                       t_insert(resp, response.body);
+               if request.method ~= "HEAD" then
+                       t_insert(resp, body);
                end
        else
                -- Response we have is just a string (the body)
-               log("debug", "Sending response to %s: %s", request.id or "<none>", response or "<none>");
+               log("debug", "Sending 200 response to %s", request.id or "<none>");
                
                resp = { "HTTP/1.0 200 OK\r\n" };
                t_insert(resp, "Connection: close\r\n");
@@ -88,9 +89,6 @@ local function call_callback(request, err)
                end
                
                callback = (request.server and request.server.handlers[base]) or default_handler;
-               if callback == default_handler then
-                       log("debug", "Default callback for this request (base: "..tostring(base)..")")
-               end
        end
        if callback then
                if err then
@@ -105,7 +103,7 @@ local function call_callback(request, err)
                if response then
                        if response == true and not request.destroyed then
                                -- Keep connection open, we will reply later
-                               log("warn", "Request %s left open, on_destroy is %s", request.id, tostring(request.on_destroy));
+                               log("debug", "Request %s left open, on_destroy is %s", request.id, tostring(request.on_destroy));
                        elseif response ~= true then
                                -- Assume response
                                send_response(request, response);
@@ -250,13 +248,17 @@ function new(params)
        end
 end
 
-function new_from_config(ports, handle_request)
+function set_default_handler(handler)
+       default_handler = handler;
+end
+
+function new_from_config(ports, default_base, handle_request)
        for _, options in ipairs(ports) do
-               local port, base, ssl, interface = 5280, "http-bind", false, nil;
+               local port, base, ssl, interface = 5280, default_base, false, nil;
                if type(options) == "number" then
                        port = options;
                elseif type(options) == "table" then
-                       port, base, ssl, interface = options.port or 5280, options.path or "http-bind", options.ssl or false, options.interface;
+                       port, base, ssl, interface = options.port or 5280, options.path or default_base, options.ssl or false, options.interface;
                elseif type(options) == "string" then
                        base = options;
                end