Merge Tobias->trunk
[prosody.git] / net / httpserver.lua
index 654025bab145190caca2f7f7678c2b9e9dd7679b..4c1200acd7df2bf282dda7565a53c97cb8890f22 100644 (file)
@@ -1,6 +1,6 @@
 -- Prosody IM
--- Copyright (C) 2008-2009 Matthew Wild
--- Copyright (C) 2008-2009 Waqas Hussain
+-- Copyright (C) 2008-2010 Matthew Wild
+-- Copyright (C) 2008-2010 Waqas Hussain
 -- 
 -- This project is MIT/X11 licensed. Please see the
 -- COPYING file in the source package for more information.
@@ -10,6 +10,7 @@
 local socket = require "socket"
 local server = require "net.server"
 local url_parse = require "socket.url".parse;
+local httpstream_new = require "util.httpstream".new;
 
 local connlisteners_start = require "net.connlisteners".start;
 local connlisteners_get = require "net.connlisteners".get;
@@ -23,9 +24,6 @@ local urlencode = function (s) return s and (s:gsub("%W", function (c) return st
 
 local log = require "util.logger".init("httpserver");
 
--- TODO: Should we read this from /etc/mime.types if it exists? (startup time...?)
-local mime_map = { html = "text/html", txt = "plain/text; charset=utf-8", js = "text/javascript" };
-
 local http_servers = {};
 
 module "httpserver"
@@ -33,51 +31,44 @@ module "httpserver"
 local default_handler;
 
 local function expectbody(reqt)
-    return reqt.method == "POST";
+       return reqt.method == "POST";
 end
 
 local function send_response(request, response)
        -- Write status line
        local resp;
-       if response.body then
-               local body = tostring(response.body);
+       if response.body or response.headers then
+               local body = response.body and tostring(response.body);
                log("debug", "Sending response to %s", request.id);
-               resp = { "HTTP/1.0 ", response.status or "200 OK", "\r\n"};
+               resp = { "HTTP/1.0 "..(response.status or "200 OK").."\r\n" };
                local h = response.headers;
                if h then
                        for k, v in pairs(h) do
-                               t_insert(resp, k);
-                               t_insert(resp, ": ");
-                               t_insert(resp, v);
-                               t_insert(resp, "\r\n");
+                               t_insert(resp, k..": "..v.."\r\n");
                        end
                end
-               if not (h and h["Content-Length"]) then
-                       t_insert(resp, "Content-Length: ");
-                       t_insert(resp, #body);
-                       t_insert(resp, "\r\n");
+               if body and not (h and h["Content-Length"]) then
+                       t_insert(resp, "Content-Length: "..#body.."\r\n");
                end
                t_insert(resp, "\r\n");
                
-               if request.method ~= "HEAD" then
+               if body and request.method ~= "HEAD" then
                        t_insert(resp, body);
                end
+               request.write(t_concat(resp));
        else
                -- Response we have is just a string (the body)
                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");
-               t_insert(resp, "Content-Type: ");
-               t_insert(resp, mime_map[request.url.path:match("%.(%w+)")] or "application/octet-stream");
-               t_insert(resp, "\r\n");
-               t_insert(resp, "Content-Length: ");
-               t_insert(resp, #response);
-               t_insert(resp, "\r\n\r\n");
+               local resp = "HTTP/1.0 200 OK\r\n"
+                       .. "Connection: close\r\n"
+                       .. "Content-Type: text/html\r\n"
+                       .. "Content-Length: "..#response.."\r\n"
+                       .. "\r\n"
+                       .. response;
                
-               t_insert(resp, response);
+               request.write(resp);
        end
-       request.write(t_concat(resp));
        if not request.stayopen then
                request:destroy();
        end
@@ -124,98 +115,34 @@ local function call_callback(request, err)
 end
 
 local function request_reader(request, data, startpos)
-       if not data then
-               if request.body then
+       if not request.parser then
+               local function success_cb(r)
+                       for k,v in pairs(r) do request[k] = v; end
+                       request.url = url_parse(request.path);
+                       request.body = { request.body };
                        call_callback(request);
-               else
-                       -- Error.. connection was closed prematurely
-                       call_callback(request, "connection-closed");
-               end
-               -- Here we force a destroy... the connection is gone, so we can't reply later
-               destroy_request(request);
-               return;
-       end
-       if request.state == "body" then
-               log("debug", "Reading body...")
-               if not request.body then request.body = {}; request.havebodylength, request.bodylength = 0, tonumber(request.headers["content-length"]); end
-               if startpos then
-                       data = data:sub(startpos, -1)
-               end
-               t_insert(request.body, data);
-               if request.bodylength then
-                       request.havebodylength = request.havebodylength + #data;
-                       if request.havebodylength >= request.bodylength then
-                               -- We have the body
-                               call_callback(request);
-                       end
-               end
-       elseif request.state == "headers" then
-               log("debug", "Reading headers...")
-               local pos = startpos;
-               local headers = request.headers or {};
-               for line in data:gmatch("(.-)\r\n") do
-                       startpos = (startpos or 1) + #line + 2;
-                       local k, v = line:match("(%S+): (.+)");
-                       if k and v then
-                               headers[k:lower()] = v;
---                             log("debug", "Header: "..k:lower().." = "..v);
-                       elseif #line == 0 then
-                               request.headers = headers;
-                               break;
-                       else
-                               log("debug", "Unhandled header line: "..line);
-                       end
-               end
-               
-               if not expectbody(request) then 
-                       call_callback(request);
-                       return;
                end
-               
-               -- Reached the end of the headers
-               request.state = "body";
-               if #data > startpos then
-                       return request_reader(request, data:sub(startpos, -1));
-               end
-       elseif request.state == "request" then
-               log("debug", "Reading request line...")
-               local method, path, http, linelen = data:match("^(%S+) (%S+) HTTP/(%S+)\r\n()", startpos);
-               if not method then
-                       return call_callback(request, "invalid-status-line");
-               end
-               
-               request.method, request.path, request.httpversion = method, path, http;
-               
-               request.url = url_parse(request.path);
-               
-               log("debug", method.." request for "..tostring(request.path) .. " on port "..request.handler.serverport());
-               
-               if request.onlystatus then
-                       if not call_callback(request) then
-                               return;
-                       end
-               end
-               
-               request.state = "headers";
-               
-               if #data > linelen then
-                       return request_reader(request, data:sub(linelen, -1));
+               local function error_cb(r)
+                       call_callback(request, r or "connection-closed");
+                       destroy_request(request);
                end
+               request.parser = httpstream_new(success_cb, error_cb);
        end
+       request.parser:feed(data);
 end
 
 -- The default handler for requests
 default_handler = function (method, body, request)
-       log("debug", method.." request for "..tostring(request.path) .. " on port "..request.handler.serverport());
-       return { status = "404 Not Found", 
+       log("debug", method.." request for "..tostring(request.path) .. " on port "..request.handler:serverport());
+       return { status = "404 Not Found",
                        headers = { ["Content-Type"] = "text/html" },
                        body = "<html><head><title>Page Not Found</title></head><body>Not here :(</body></html>" };
 end
 
 
 function new_request(handler)
-       return { handler = handler, conn = handler.socket, 
-                       write = function (...) return handler:write(...); end, state = "request", 
+       return { handler = handler, conn = handler,
+                       write = function (...) return handler:write(...); end, state = "request",
                        server = http_servers[handler:serverport()],
                        send = send_response,
                        destroy = destroy_request,
@@ -234,9 +161,9 @@ function destroy_request(request)
                else
                        log("debug", "Request has no destroy callback");
                end
-               request.handler.close()
+               request.handler:close()
                if request.conn then
-                       listener.ondisconnect(request.handler, "closed");
+                       listener.ondisconnect(request.conn, "closed");
                end
        end
 end
@@ -263,6 +190,7 @@ function new_from_config(ports, handle_request, default_options)
                log("warn", "Old syntax of httpserver.new_from_config being used to register %s", handle_request);
                handle_request, default_options = default_options, { base = handle_request };
        end
+       ports = ports or {5280};
        for _, options in ipairs(ports) do
                local port = default_options.port or 5280;
                local base = default_options.base;
@@ -282,10 +210,11 @@ function new_from_config(ports, handle_request, default_options)
                if ssl then
                        ssl.mode = "server";
                        ssl.protocol = "sslv23";
+                       ssl.options = "no_sslv2";
                end
                
-               new{ port = port, interface = interface, 
-                       base = base, handler = handle_request, 
+               new{ port = port, interface = interface,
+                       base = base, handler = handle_request,
                        ssl = ssl, type = (ssl and "ssl") or "tcp" };
        end
 end