Debians commandline package manager https://git.kalnischkies.de/apt/atom?h=1.0.9.5 2014-12-22T13:23:39Z 2014-12-22T13:23:39Z David Kalnischkies david@kalnischkies.de 2014-11-29T16:59:52Z urn:sha1:92e8c1ff287ab829de825e00cdf94744e699ff97 Real webservers (like apache) actually send an error page with a 416 response, but our client didn't expect it leaving the page on the socket to be parsed as response for the next request (http) or as file content (https), which isn't what we want at all… Symptom is a "Bad header line" as html usually doesn't parse that well to an http-header. This manifests itself e.g. if we have a complete file (or larger) in partial/ which isn't discarded by If-Range as the server doesn't support it (or it is just newer, think: mirror rotation). It is a sort-of regression of 78c72d0ce22e00b194251445aae306df357d5c1a, which removed the filesize - 1 trick, but this had its own problems… To properly test this our webserver gains the ability to reply with transfer-encoding: chunked as most real webservers will use it to send the dynamically generated error pages. (The tests and their binary helpers had to be slightly modified to apply, but the patch to fix the issue itself is unchanged.) Closes: 768797 2014-09-05T14:24:32Z Michael Vogt mvo@ubuntu.com 2014-09-05T14:24:32Z urn:sha1:9622b2111095c3fc705ec0615d27fe403e18c3b8 Prefix all answers with the URL that the answer is for. This helps when debugging and pipeline is enabled. 2014-03-13T12:58:45Z David Kalnischkies david@kalnischkies.de 2014-03-05T23:33:10Z urn:sha1:a02db58fd50ef7fc2f0284852c6b3f98e458a232 Git-Dch: Ignore Reported-By: gcc -Wsuggest-attribute={pure,const,noreturn} 2014-03-13T12:58:45Z David Kalnischkies david@kalnischkies.de 2014-03-05T21:11:25Z urn:sha1:453b82a388013e522b3a1b9fcd6ed0810dab1f4f Beside being a bit cleaner it hopefully also resolves oddball problems I have with high levels of parallel jobs. Git-Dch: Ignore Reported-By: iwyu (include-what-you-use) 2014-03-13T12:58:45Z David Kalnischkies david@kalnischkies.de 2014-02-27T00:26:29Z urn:sha1:3de8f956d5fcf023ab65f88579cd77121694d872 server.cc: In member function ‘bool ServerState::HeaderLine(std::string)’: server.cc:198:72: warning: format ‘%llu’ expects argument of type ‘long long unsigned int*’, but argument 3 has type ‘long long int*’ [-Wformat=] else if (sscanf(Val.c_str(),"bytes %llu-%*u/%llu",&StartPos,&Size) != 2) Git-Dch: Ignore Reported-By: gcc -Wpedantic 2014-03-13T12:58:45Z David Kalnischkies david@kalnischkies.de 2014-02-27T00:20:53Z urn:sha1:d3e8fbb395f57954acd7a2095f02ce530a05ec6a Git-Dch: Ignore Reported-By: gcc -Wpedantic 2014-01-16T21:19:49Z David Kalnischkies david@kalnischkies.de 2014-01-16T21:19:49Z urn:sha1:9ce3cfc9309c55cc01018c88c1ca82779fd74431 The most "visible" change is from utime to utimensat/futimens as the first one isn't part of POSIX anymore. Reported-By: cppcheck Git-Dch: Ignore 2013-10-01T09:17:02Z David Kalnischkies kalnischkies@gmail.com 2013-09-30T14:41:16Z urn:sha1:fd46d30571eb240ec3aae792e7a56061ede50524 Servers might respond with a complete file either because they don't support Ranges at all or the If-Range condition isn't statisfied, so we have to parse the headers curl gets ourself to seek or truncate the file we have so far. This also finially adds the testcase testing a bunch of partial situations for both, http and https - which is now all green. Closes: 617643, 667699 LP: 1157943 2013-10-01T09:06:56Z David Kalnischkies kalnischkies@gmail.com 2013-09-17T20:35:44Z urn:sha1:7330f4df8b31e66f6557bf49c9c90ad9a73ff459 No effective behavior change, just shuffling big junks of code between methods and classes to split them into those strongly related to our client implementation and those implementing HTTP. The idea is to get HTTPS to a point in which most of the implementation can be shared even though the client implementations itself is completely different. This isn't anywhere near yet though, but it should beenough to reuse at least a few lines from http in https now. Git-Dch: Ignore