Do not try to write duplicate response on error in vibe.web.rest. Fixes

#821.
vibe-d · Sep 12, 2014 · 00caa4d · 00caa4d
1 parent 54ea846
commit 00caa4d
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/source/vibe/web/rest.d b/source/vibe/web/rest.d
@@ -550,11 +550,13 @@ private HTTPServerRequestDelegate jsonMethodHandler(T, string method, alias Func
 				res.writeJsonBody(ret);
 			}
 		} catch (HTTPStatusException e) {
-			res.writeJsonBody([ "statusMessage": e.msg ], e.status);
+			if (res.headerWritten) logDebug("Response already started when a HTTPStatusException was thrown. Client will not receive the proper error code (%s)!", e.status);
+			else res.writeJsonBody([ "statusMessage": e.msg ], e.status);
 		} catch (Exception e) {
 			// TODO: better error description!
 			logDebug("REST handler exception: %s", e.toString());
-			res.writeJsonBody(
+			if (res.headerWritten) logDebug("Response already started. Client will not receive an error code!");
+			else res.writeJsonBody(
 				[ "statusMessage": e.msg, "statusDebugMessage": sanitizeUTF8(cast(ubyte[])e.toString()) ],
 				HTTPStatus.internalServerError
 			);