apt/methods/http.cc, branch 1.0.9.5Debians commandline package managerhttps://git.kalnischkies.de/apt/atom?h=1.0.9.52014-12-22T13:23:39Zdispose http(s) 416 error page as non-content2014-12-22T13:23:39ZDavid Kalnischkiesdavid@kalnischkies.de2014-11-29T16:59:52Zurn: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
Make Proxy-Auto-Detect check for each host2014-09-02T14:02:37ZMichael Vogtmvo@ubuntu.com2014-09-02T13:50:19Zurn:sha1:c6ee61eab54edf6cc3fbe118d304d72a860e1451
When doing Acquire::http{,s}::Proxy-Auto-Detect, run the auto-detect
command for each host instead of only once. This should make using
"proxy" from libproxy-tools feasible which can then be used for PAC
style or other proxy configurations.
Closes: #759264
methods/http.cc: use Req.str() in debug output2014-06-24T13:45:09ZMichael Vogtmvo@debian.org2014-06-24T13:45:09Zurn:sha1:7b734b09f6bd9356e4622aee64bd2e5e43554570build http request in a stringstream2014-04-26T07:51:05ZDavid Kalnischkiesdavid@kalnischkies.de2014-04-24T08:20:45Zurn:sha1:b123b0ba77936a82d7964bebf01693acf94cae1f
beside reducing code a bit, it avoids oddball problems while building
the string and doesn't trigger static analyse warnings.
follow method attribute suggestions by gcc2014-03-13T12:58:45ZDavid Kalnischkiesdavid@kalnischkies.de2014-03-05T23:33:10Zurn:sha1:a02db58fd50ef7fc2f0284852c6b3f98e458a232
Git-Dch: Ignore
Reported-By: gcc -Wsuggest-attribute={pure,const,noreturn}
cleanup headers and especially #includes everywhere2014-03-13T12:58:45ZDavid Kalnischkiesdavid@kalnischkies.de2014-03-05T21:11:25Zurn: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)
StartPos is always positive for http/https2014-03-13T12:58:45ZDavid Kalnischkiesdavid@kalnischkies.de2014-02-27T00:26:29Zurn: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
warning: extra ‘;’ [-Wpedantic]2014-03-13T12:58:45ZDavid Kalnischkiesdavid@kalnischkies.de2014-02-27T00:20:53Zurn:sha1:d3e8fbb395f57954acd7a2095f02ce530a05ec6a
Git-Dch: Ignore
Reported-By: gcc -Wpedantic
Fix typos in documentation (codespell)2014-02-22T17:34:33ZMichael Vogtmvo@debian.org2014-02-22T17:34:33Zurn:sha1:1e3f4083db29bba600b9725e9456b0e140975c99fix various style/performance warnings in rred2014-01-29T23:16:49ZDavid Kalnischkiesdavid@kalnischkies.de2014-01-29T23:11:05Zurn:sha1:25d99f3b42ada24679ddc1d911530425acc8e475
Reported-By: cppcheck
Git-Dch: Ignore