X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fhttp%2Fserver.cpp;h=f985bf994526bf871844b97cfd263f2ee56ee2ef;hb=d20870655aeca0614d9fc315fe5f9893ebdc7aa3;hp=ed3a2b9288339830ee89f0b56585960ddb24d398;hpb=5f6c5ae5527619f7ffc82a69cf73d3fc7347e51f;p=libs%2Fnet.git diff --git a/source/http/server.cpp b/source/http/server.cpp index ed3a2b9..f985bf9 100644 --- a/source/http/server.cpp +++ b/source/http/server.cpp @@ -1,10 +1,13 @@ #include +#include #include #include +#include #include #include #include #include +#include #include "request.h" #include "response.h" #include "server.h" @@ -23,6 +26,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 +65,12 @@ 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::cancel_keepalive(Response &resp) +{ + get_client_by_response(resp).keepalive = false; } void Server::data_available() @@ -82,11 +92,19 @@ void Server::client_data_available(Client &cl) break; } - char rbuf[4096]; - unsigned len = cl.sock->read(rbuf, sizeof(rbuf)); - if(cl.stale) + try + { + char rbuf[4096]; + unsigned len = cl.sock->read(rbuf, sizeof(rbuf)); + if(cl.stale) + return; + cl.in_buf.append(rbuf, len); + } + catch(const exception &) + { + cl.stale = true; return; - cl.in_buf.append(rbuf, len); + } RefPtr response; if(!cl.request) @@ -98,7 +116,7 @@ void Server::client_data_available(Client &cl) cl.request = new Request(Request::parse(cl.in_buf)); string addr_str = cl.sock->get_peer_address().str(); - unsigned colon = addr_str.find(':'); + string::size_type colon = addr_str.find(':'); cl.request->set_header("-Client-Host", addr_str.substr(0, colon)); if(cl.request->get_method()!="GET" && cl.request->get_method()!="POST") @@ -115,22 +133,23 @@ void Server::client_data_available(Client &cl) catch(const exception &e) { response = new Response(BAD_REQUEST); - response->add_content(e.what()); + response->add_content(format("An error occurred while parsing request headers:\ntype: %s\nwhat: %s", + Debug::demangle(typeid(e).name()), e.what())); } cl.in_buf = string(); } } else { - len = cl.request->parse_content(cl.in_buf); + unsigned len = cl.request->parse_content(cl.in_buf); cl.in_buf.erase(0, len); } - bool keepalive = false; if(cl.request && cl.request->is_complete() && !response) { + cl.keepalive = false; if(cl.request->has_header("Connection")) - keepalive = (cl.request->get_header("Connection")=="keep-alive"); + cl.keepalive = !strcasecmp(cl.request->get_header("Connection"), "keep-alive"); response = new Response(NONE); try @@ -156,25 +175,42 @@ void Server::client_data_available(Client &cl) responses.erase(cl.response); cl.response = 0; response = new Response(INTERNAL_ERROR); - response->add_content(e.what()); + response->add_content(format("An error occurred while processing the request:\ntype: %s\nwhat: %s", + Debug::demangle(typeid(e).name()), e.what())); } } if(response) + send_response(cl, *response); +} + +void Server::send_response(Client &cl, Response &resp) +{ + if(cl.keepalive) + resp.set_header("Connection", "keep-alive"); + + try { - cl.sock->write(response->str()); - if(keepalive) - { - delete cl.request; - cl.request = 0; - delete cl.response; - cl.response = 0; - } - else - { - cl.sock->shutdown(IO::M_WRITE); - cl.stale = true; - } + cl.sock->write(resp.str()); + } + catch(const exception &) + { + cl.stale = true; + return; + } + + cl.async = false; + if(cl.keepalive) + { + delete cl.request; + cl.request = 0; + delete cl.response; + cl.response = 0; + } + else + { + cl.sock->shutdown(IO::M_WRITE); + cl.stale = true; } } @@ -193,6 +229,7 @@ Server::Client::Client(RefPtr s): sock(s), request(0), response(0), + keepalive(false), async(false), stale(false) { }