diff options
| author | Edgar Fuß <ef@math.uni-bonn.de> | 2016-11-11 10:20:23 +0100 |
|---|---|---|
| committer | Julian Andres Klode <jak@debian.org> | 2017-02-22 16:53:45 +0100 |
| commit | 4ff096a524a4ad276aa785d3886875cf68b8c3fe (patch) | |
| tree | 247289e4e739692410f593a3192805bdcc80e6d0 | |
| parent | 52067bd0a9e23642b7fa791fb63f4b69cafceb36 (diff) | |
| download | apt-4ff096a524a4ad276aa785d3886875cf68b8c3fe.tar.gz | |
http: clear content before reporting the failure
[Comment from commiter:] I have the feeling that the issue itself is
fixed for a while already as nowadays we have testcases involving a
webserver closing the connection on error (look for "closeOnError") and
no even remotely recent reports about it, but moving the content
clearance above the failure report is a valid change and shouldn't hurt.
Closes: #465572
(cherry picked from commit 324bb34d77a43d1be411c402b2e11f588194439a)
| -rw-r--r-- | methods/server.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/methods/server.cc b/methods/server.cc index af7276bad..0408dddfd 100644 --- a/methods/server.cc +++ b/methods/server.cc @@ -770,8 +770,8 @@ int ServerMethod::Loop() // We need to flush the data, the header is like a 404 w/ error text case ERROR_WITH_CONTENT_PAGE: { - Fail(); Server->RunDataToDevNull(); + Fail(); break; } |