X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=server.cpp;h=8c3382d4e8c6506d4c6a99aa686c2bc5121fa436;hb=4f4e1384b4299611924a39f59f536b4964806135;hp=02efdbbdb6c165612f2ac45219072bbfbd414982;hpb=d1ad2cf60ca694f742e2061a596bc27c7e58f5c8;p=cubemap

diff --git a/server.cpp b/server.cpp
index 02efdbb..8c3382d 100644
--- a/server.cpp
+++ b/server.cpp
@@ -14,6 +14,7 @@
 #include <utility>
 #include <vector>
 
+#include "accesslog.h"
 #include "log.h"
 #include "markpool.h"
 #include "mutexlock.h"
@@ -24,6 +25,8 @@
 
 using namespace std;
 
+extern AccessLogThread *access_log;
+
 Server::Server()
 {
 	pthread_mutex_init(&mutex, NULL);
@@ -31,20 +34,26 @@ Server::Server()
 
 	epoll_fd = epoll_create(1024);  // Size argument is ignored.
 	if (epoll_fd == -1) {
-		perror("epoll_fd");
+		log_perror("epoll_fd");
 		exit(1);
 	}
 }
 
 Server::~Server()
 {
+	for (map<string, Stream *>::iterator stream_it = streams.begin();
+	     stream_it != streams.end();
+	     ++stream_it) {
+		delete stream_it->second;
+	}
+
 	int ret;
 	do {
 		ret = close(epoll_fd);
 	} while (ret == -1 && errno == EINTR);
 
 	if (ret == -1) {
-		perror("close(epoll_fd)");
+		log_perror("close(epoll_fd)");
 	}
 }
 
@@ -72,7 +81,7 @@ void Server::do_work()
 			continue;
 		}
 		if (nfds == -1) {
-			perror("epoll_wait");
+			log_perror("epoll_wait");
 			exit(1);
 		}
 
@@ -141,7 +150,7 @@ void Server::add_client(int sock)
 	ev.events = EPOLLIN | EPOLLET | EPOLLRDHUP;
 	ev.data.u64 = reinterpret_cast<uint64_t>(&clients[sock]);
 	if (epoll_ctl(epoll_fd, EPOLL_CTL_ADD, sock, &ev) == -1) {
-		perror("epoll_ctl(EPOLL_CTL_ADD)");
+		log_perror("epoll_ctl(EPOLL_CTL_ADD)");
 		exit(1);
 	}
 
@@ -173,7 +182,7 @@ void Server::add_client_from_serialized(const ClientProto &client)
 	ev.data.u64 = 0;  // Keep Valgrind happy.
 	ev.data.u64 = reinterpret_cast<uint64_t>(client_ptr);
 	if (epoll_ctl(epoll_fd, EPOLL_CTL_ADD, client.sock(), &ev) == -1) {
-		perror("epoll_ctl(EPOLL_CTL_ADD)");
+		log_perror("epoll_ctl(EPOLL_CTL_ADD)");
 		exit(1);
 	}
 
@@ -254,7 +263,7 @@ read_request_again:
 			return;
 		}
 		if (ret == -1) {
-			perror("read");
+			log_perror("read");
 			close_client(client);
 			return;
 		}
@@ -268,7 +277,7 @@ read_request_again:
 	
 		switch (status) {
 		case RP_OUT_OF_SPACE:
-			log(WARNING, "fd %d sent overlong request!", client->sock);
+			log(WARNING, "[%s] Client sent overlong request!", client->remote_addr.c_str());
 			close_client(client);
 			return;
 		case RP_NOT_FINISHED_YET:
@@ -276,7 +285,7 @@ read_request_again:
 			// See if there's more data for us.
 			goto read_request_again;
 		case RP_EXTRA_DATA:
-			log(WARNING, "fd %d had junk data after request!", client->sock);
+			log(WARNING, "[%s] Junk data after request!", client->remote_addr.c_str());
 			close_client(client);
 			return;
 		case RP_FINISHED:
@@ -316,7 +325,7 @@ sending_header_or_error_again:
 
 		if (ret == -1) {
 			// Error! Postcondition #1.
-			perror("write");
+			log_perror("write");
 			close_client(client);
 			return;
 		}
@@ -357,8 +366,8 @@ sending_data_again:
 			return;
 		}
 		if (bytes_to_send > stream->backlog_size) {
-			log(WARNING, "fd %d lost %lld bytes, maybe too slow connection",
-			        client->sock,
+			log(WARNING, "[%s] Client lost %lld bytes, maybe too slow connection",
+			        client->remote_addr.c_str(),
 			        (long long int)(bytes_to_send - stream->backlog_size));
 			client->stream_pos = stream->bytes_received - stream->backlog_size;
 			client->bytes_lost += bytes_to_send - stream->backlog_size;
@@ -387,7 +396,7 @@ sending_data_again:
 		}
 		if (ret == -1) {
 			// Error, close; postcondition #1.
-			perror("sendfile");
+			log_perror("sendfile");
 			close_client(client);
 			return;
 		}
@@ -435,7 +444,7 @@ int Server::parse_request(Client *client)
 	}
 	if (setsockopt(client->sock, SOL_SOCKET, SO_MARK, &client->fwmark, sizeof(client->fwmark)) == -1) {                          
 		if (client->fwmark != 0) {
-			perror("setsockopt(SO_MARK)");
+			log_perror("setsockopt(SO_MARK)");
 		}
 	}
 	client->request.clear();
@@ -455,7 +464,7 @@ void Server::construct_header(Client *client)
 	ev.data.u64 = reinterpret_cast<uint64_t>(client);
 
 	if (epoll_ctl(epoll_fd, EPOLL_CTL_MOD, client->sock, &ev) == -1) {
-		perror("epoll_ctl(EPOLL_CTL_MOD)");
+		log_perror("epoll_ctl(EPOLL_CTL_MOD)");
 		exit(1);
 	}
 }
@@ -475,7 +484,7 @@ void Server::construct_error(Client *client, int error_code)
 	ev.data.u64 = reinterpret_cast<uint64_t>(client);
 
 	if (epoll_ctl(epoll_fd, EPOLL_CTL_MOD, client->sock, &ev) == -1) {
-		perror("epoll_ctl(EPOLL_CTL_MOD)");
+		log_perror("epoll_ctl(EPOLL_CTL_MOD)");
 		exit(1);
 	}
 }
@@ -490,7 +499,7 @@ void delete_from(vector<T> *v, T elem)
 void Server::close_client(Client *client)
 {
 	if (epoll_ctl(epoll_fd, EPOLL_CTL_DEL, client->sock, NULL) == -1) {
-		perror("epoll_ctl(EPOLL_CTL_DEL)");
+		log_perror("epoll_ctl(EPOLL_CTL_DEL)");
 		exit(1);
 	}
 
@@ -504,6 +513,9 @@ void Server::close_client(Client *client)
 		}
 	}
 
+	// Log to access_log.
+	access_log->write(client->get_stats());
+
 	// Bye-bye!
 	int ret;
 	do {
@@ -511,7 +523,7 @@ void Server::close_client(Client *client)
 	} while (ret == -1 && errno == EINTR);
 
 	if (ret == -1) {
-		perror("close");
+		log_perror("close");
 	}
 
 	clients.erase(client->sock);