diff options
| author | Edgar Fuß <ef@math.uni-bonn.de> | 2016-11-11 10:20:23 +0100 |
|---|---|---|
| committer | David Kalnischkies <david@kalnischkies.de> | 2016-11-11 23:39:50 +0100 |
| commit | 324bb34d77a43d1be411c402b2e11f588194439a (patch) | |
| tree | 095e233d196b835dc0e9adfb89e1dc5396ac96aa /methods | |
| parent | e2c8c825a5470e33c25d00e07de188d0e03922c8 (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
Diffstat (limited to 'methods')
| -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; } |