summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEdgar Fuß <ef@math.uni-bonn.de>2016-11-11 10:20:23 +0100
committerJulian Andres Klode <jak@debian.org>2017-02-22 16:53:45 +0100
commit4ff096a524a4ad276aa785d3886875cf68b8c3fe (patch)
tree247289e4e739692410f593a3192805bdcc80e6d0
parent52067bd0a9e23642b7fa791fb63f4b69cafceb36 (diff)
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.cc2
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;
}