X-Git-Url: https://git.sesse.net/?p=cubemap;a=blobdiff_plain;f=httpinput.cpp;h=da672edc3689a14cb4707f29f060d9db73183bad;hp=9d39774b70a18188fba21020421261ab56faafc5;hb=b9f87872a42f32818805f3c2520555f2d6e2928d;hpb=f87901a18647fb66d766a1db55589fc753f46008 diff --git a/httpinput.cpp b/httpinput.cpp index 9d39774..da672ed 100644 --- a/httpinput.cpp +++ b/httpinput.cpp @@ -1,39 +1,34 @@ +#include +#include +#include +#include +#include +#include #include +#include #include -#include -#include -#include -#include -#include -#include -#include #include -#include #include -#include -#include -#include -#include -#include -#include +#include #include +#include +#include +#include -#include "metacube.h" -#include "mutexlock.h" #include "httpinput.h" -#include "server.h" -#include "serverpool.h" +#include "log.h" +#include "metacube.h" #include "parse.h" -#include "version.h" +#include "serverpool.h" #include "state.pb.h" +#include "version.h" using namespace std; extern ServerPool *servers; -HTTPInput::HTTPInput(const string &stream_id, const string &url) +HTTPInput::HTTPInput(const string &url) : state(NOT_CONNECTED), - stream_id(stream_id), url(url), has_metacube_header(false), sock(-1) @@ -42,7 +37,6 @@ HTTPInput::HTTPInput(const string &stream_id, const string &url) HTTPInput::HTTPInput(const InputProto &serialized) : state(State(serialized.state())), - stream_id(serialized.stream_id()), url(serialized.url()), request(serialized.request()), request_bytes_sent(serialized.request_bytes_sent()), @@ -56,6 +50,13 @@ HTTPInput::HTTPInput(const InputProto &serialized) string protocol; parse_url(url, &protocol, &host, &port, &path); // Don't care if it fails. + + // Older versions stored the extra \r\n in the HTTP header. + // Strip it if we find it. + if (http_header.size() >= 4 && + memcmp(http_header.data() + http_header.size() - 4, "\r\n\r\n", 4) == 0) { + http_header.resize(http_header.size() - 2); + } } void HTTPInput::close_socket() @@ -66,7 +67,7 @@ void HTTPInput::close_socket() } while (ret == -1 && errno == EINTR); if (ret == -1) { - perror("close()"); + log_perror("close()"); } } @@ -74,7 +75,6 @@ InputProto HTTPInput::serialize() const { InputProto serialized; serialized.set_state(state); - serialized.set_stream_id(stream_id); serialized.set_url(url); serialized.set_request(request); serialized.set_request_bytes_sent(request_bytes_sent); @@ -91,12 +91,14 @@ int HTTPInput::lookup_and_connect(const string &host, const string &port) addrinfo *ai; int err = getaddrinfo(host.c_str(), port.c_str(), NULL, &ai); if (err == -1) { - fprintf(stderr, "WARNING: Lookup of '%s' failed (%s).\n", - host.c_str(), gai_strerror(err)); + log(WARNING, "[%s] Lookup of '%s' failed (%s).", + url.c_str(), host.c_str(), gai_strerror(err)); freeaddrinfo(ai); return -1; } + addrinfo *base_ai = ai; + // Connect to everything in turn until we have a socket. while (ai && !should_stop) { int sock = socket(ai->ai_family, SOCK_STREAM, IPPROTO_TCP); @@ -110,17 +112,26 @@ int HTTPInput::lookup_and_connect(const string &host, const string &port) } while (err == -1 && errno == EINTR); if (err != -1) { - freeaddrinfo(ai); + freeaddrinfo(base_ai); return sock; } + do { + err = close(sock); + } while (err == -1 && errno == EINTR); + + if (err == -1) { + log_perror("close"); + // Can still continue. + } + ai = ai->ai_next; } // Give the last one as error. - fprintf(stderr, "WARNING: Connect to '%s' failed (%s)\n", - host.c_str(), strerror(errno)); - freeaddrinfo(ai); + log(WARNING, "[%s] Connect to '%s' failed (%s)", + url.c_str(), host.c_str(), strerror(errno)); + freeaddrinfo(base_ai); return -1; } @@ -128,21 +139,21 @@ bool HTTPInput::parse_response(const std::string &request) { vector lines = split_lines(response); if (lines.empty()) { - fprintf(stderr, "WARNING: Empty HTTP response from input.\n"); + log(WARNING, "[%s] Empty HTTP response from input.", url.c_str()); return false; } vector first_line_tokens = split_tokens(lines[0]); if (first_line_tokens.size() < 2) { - fprintf(stderr, "WARNING: Malformed response line '%s' from input.\n", - lines[0].c_str()); + log(WARNING, "[%s] Malformed response line '%s' from input.", + url.c_str(), lines[0].c_str()); return false; } int response = atoi(first_line_tokens[1].c_str()); if (response != 200) { - fprintf(stderr, "WARNING: Non-200 response '%s' from input.\n", - lines[0].c_str()); + log(WARNING, "[%s] Non-200 response '%s' from input.", + url.c_str(), lines[0].c_str()); return false; } @@ -150,8 +161,8 @@ bool HTTPInput::parse_response(const std::string &request) for (size_t i = 1; i < lines.size(); ++i) { size_t split = lines[i].find(":"); if (split == string::npos) { - fprintf(stderr, "WARNING: Ignoring malformed HTTP response line '%s'\n", - lines[i].c_str()); + log(WARNING, "[%s] Ignoring malformed HTTP response line '%s'", + url.c_str(), lines[i].c_str()); continue; } @@ -189,6 +200,11 @@ bool HTTPInput::parse_response(const std::string &request) } } + // Set “Connection: close”. + // TODO: Make case-insensitive. + parameters.erase("Connection"); + parameters.insert(make_pair("Connection", "close")); + // Construct the new HTTP header. http_header = "HTTP/1.0 200 OK\r\n"; for (multimap::iterator it = parameters.begin(); @@ -196,8 +212,10 @@ bool HTTPInput::parse_response(const std::string &request) ++it) { http_header.append(it->first + ": " + it->second + "\r\n"); } - http_header.append("\r\n"); - servers->set_header(stream_id, http_header); + + for (size_t i = 0; i < stream_ids.size(); ++i) { + servers->set_header(stream_ids[i], http_header, ""); + } return true; } @@ -218,7 +236,7 @@ void HTTPInput::do_work() continue; } if (nfds == -1) { - perror("poll"); + log_perror("poll"); state = CLOSING_SOCKET; } } @@ -229,11 +247,14 @@ void HTTPInput::do_work() request_bytes_sent = 0; response.clear(); pending_data.clear(); + for (size_t i = 0; i < stream_ids.size(); ++i) { + servers->set_header(stream_ids[i], "", ""); + } { string protocol; // Thrown away. if (!parse_url(url, &protocol, &host, &port, &path)) { - fprintf(stderr, "Failed to parse URL '%s'\n", url.c_str()); + log(WARNING, "[%s] Failed to parse URL '%s'", url.c_str(), url.c_str()); break; } } @@ -243,7 +264,7 @@ void HTTPInput::do_work() // Yay, successful connect. Try to set it as nonblocking. int one = 1; if (ioctl(sock, FIONBIO, &one) == -1) { - perror("ioctl(FIONBIO)"); + log_perror("ioctl(FIONBIO)"); state = CLOSING_SOCKET; } else { state = SENDING_REQUEST; @@ -261,7 +282,7 @@ void HTTPInput::do_work() } while (ret == -1 && errno == EINTR); if (ret == -1) { - perror("write"); + log_perror("write"); state = CLOSING_SOCKET; continue; } @@ -283,14 +304,15 @@ void HTTPInput::do_work() } while (ret == -1 && errno == EINTR); if (ret == -1) { - perror("read"); + log_perror("read"); state = CLOSING_SOCKET; continue; } if (ret == 0) { // This really shouldn't happen... - fprintf(stderr, "Socket unexpectedly closed while reading header\n"); + log(ERROR, "[%s] Socket unexpectedly closed while reading header", + url.c_str()); state = CLOSING_SOCKET; continue; } @@ -298,7 +320,7 @@ void HTTPInput::do_work() RequestParseStatus status = wait_for_double_newline(&response, buf, ret); if (status == RP_OUT_OF_SPACE) { - fprintf(stderr, "WARNING: fd %d sent overlong response!\n", sock); + log(WARNING, "[%s] Sever sent overlong HTTP response!", url.c_str()); state = CLOSING_SOCKET; continue; } else if (status == RP_NOT_FINISHED_YET) { @@ -325,6 +347,8 @@ void HTTPInput::do_work() process_data(&extra_data[0], extra_data.size()); } + log(INFO, "[%s] Connected to '%s', receiving data.", + url.c_str(), url.c_str()); state = RECEIVING_DATA; break; } @@ -337,14 +361,15 @@ void HTTPInput::do_work() } while (ret == -1 && errno == EINTR); if (ret == -1) { - perror("read"); + log_perror("read"); state = CLOSING_SOCKET; continue; } if (ret == 0) { // This really shouldn't happen... - fprintf(stderr, "Socket unexpectedly closed while reading header\n"); + log(ERROR, "[%s] Socket unexpectedly closed while reading header", + url.c_str()); state = CLOSING_SOCKET; continue; } @@ -359,7 +384,7 @@ void HTTPInput::do_work() } while (err == -1 && errno == EINTR); if (err == -1) { - perror("close"); + log_perror("close"); } state = NOT_CONNECTED; @@ -373,7 +398,7 @@ void HTTPInput::do_work() // or the connection just got closed. // The earlier steps have already given the error message, if any. if (state == NOT_CONNECTED && !should_stop) { - fprintf(stderr, "Waiting 0.2 second and restarting...\n"); + log(INFO, "[%s] Waiting 0.2 second and restarting...", url.c_str()); usleep(200000); } } @@ -429,9 +454,13 @@ void HTTPInput::process_data(char *ptr, size_t bytes) char *inner_data = pending_data.data() + sizeof(metacube_block_header); if (flags & METACUBE_FLAGS_HEADER) { string header(inner_data, inner_data + size); - servers->set_header(stream_id, http_header + header); + for (size_t i = 0; i < stream_ids.size(); ++i) { + servers->set_header(stream_ids[i], http_header, header); + } } else { - servers->add_data(stream_id, inner_data, size); + for (size_t i = 0; i < stream_ids.size(); ++i) { + servers->add_data(stream_ids[i], inner_data, size); + } } // Consume the block. This isn't the most efficient way of dealing with things @@ -447,8 +476,8 @@ void HTTPInput::drop_pending_data(size_t num_bytes) if (num_bytes == 0) { return; } - fprintf(stderr, "Warning: Dropping %lld junk bytes from stream, maybe it is not a Metacube stream?\n", - (long long)num_bytes); + log(WARNING, "[%s] Dropping %lld junk bytes from stream, maybe it is not a Metacube stream?", + url.c_str(), (long long)num_bytes); pending_data.erase(pending_data.begin(), pending_data.begin() + num_bytes); }