Use connection keepalive on async responses too

[libs/net.git] / source / http / server.cpp

diff --git a/source/http/server.cpp b/source/http/server.cpp
index b18ed4c76b02dec85572e41cfecac26f2f186852..fbacb10accfc6a1de8c4497503c357aa457a16ac 100644 (file)
--- a/source/http/server.cpp
+++ b/source/http/server.cpp
@@ -23,6 +23,11 @@ Server::Server(unsigned port):
        sock.listen(*addr, 8);
 }
 
+// Avoid emitting sigc::signal destructor in files including server.h
+Server::~Server()
+{
+}
+
 unsigned Server::get_port() const
 {
        const Net::SockAddr &addr = sock.get_local_address();
@@ -57,10 +62,7 @@ void Server::submit_response(Response &resp)
 {
        Client &cl = get_client_by_response(resp);
        if(cl.async)
-       {
-               cl.sock->write(resp.str());
-               cl.stale = true;
-       }
+               send_response(cl, *cl.response);
 }
 
 void Server::data_available()
@@ -106,6 +108,11 @@ void Server::client_data_available(Client &cl)
                                        response = new Response(NOT_IMPLEMENTED);
                                        response->add_content("Method not implemented\n");
                                }
+                               else if(cl.request->get_path()[0]!='/')
+                               {
+                                       response = new Response(BAD_REQUEST);
+                                       response->add_content("Path must be absolute\n");
+                               }
                        }
                        catch(const exception &e)
                        {
@@ -123,6 +130,10 @@ void Server::client_data_available(Client &cl)
 
        if(cl.request && cl.request->is_complete() && !response)
        {
+               cl.keepalive = false;
+               if(cl.request->has_header("Connection"))
+                       cl.keepalive = (cl.request->get_header("Connection")=="keep-alive");
+
                response = new Response(NONE);
                try
                {
@@ -152,8 +163,23 @@ void Server::client_data_available(Client &cl)
        }
 
        if(response)
+               send_response(cl, *response);
+}
+
+void Server::send_response(Client &cl, Response &resp)
+{
+       cl.sock->write(resp.str());
+       cl.async = false;
+       if(cl.keepalive)
+       {
+               delete cl.request;
+               cl.request = 0;
+               delete cl.response;
+               cl.response = 0;
+       }
+       else
        {
-               cl.sock->write(response->str());
+               cl.sock->shutdown(IO::M_WRITE);
                cl.stale = true;
        }
 }
@@ -173,6 +199,7 @@ Server::Client::Client(RefPtr<Net::StreamSocket> s):
        sock(s),
        request(0),
        response(0),
+       keepalive(false),
        async(false),
        stale(false)
 { }