From d0eb1d75164ab218856f107ccfdae06dfd1179b4 Mon Sep 17 00:00:00 2001 From: "P. Christeas" Date: Mon, 26 Jul 2010 12:33:35 +0300 Subject: [PATCH] websrv_lib: fix handling of method exceptions. They should not consume the body (twice), pass the error to the log(..), since it may contain '%' characters. bzr revid: p_christ@hol.gr-20100726093335-wgh34fxda2nt64ze --- bin/service/websrv_lib.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/bin/service/websrv_lib.py b/bin/service/websrv_lib.py index 0022fe6daf4..b569ab57acb 100644 --- a/bin/service/websrv_lib.py +++ b/bin/service/websrv_lib.py @@ -253,8 +253,7 @@ class MultiHTTPHandler(FixSendError, HttpOptions, BaseHTTPRequestHandler): except (AuthRejectedExc, AuthRequiredExc): raise except Exception, e: - self.log_error("Could not run %s: %s" % (mname, e)) - self._get_ignore_body(fore) + self.log_error("Could not run %s: %s", mname, e) self.send_error(500, "Internal error") # may not work if method has already sent data fore.close_connection = 1