From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Sun, 23 Apr 2017 19:51:21 +0000 (+0100)
Subject: hippotatd: catch errors from request.finish in _req_cancel
X-Git-Url: https://git.distorted.org.uk/~mdw/hippotat/commitdiff_plain/b6c0b1bb37610c084ebcb477332ac693ea089bd6?hp=02a201e1e706788f355171ff979e1f82ecadfc71

hippotatd: catch errors from request.finish in _req_cancel

It is not completely clear when it is necessary to call
request.finish.  Doing so when it is not necessary has a tendency to
throw an exception.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---

diff --git a/hippotatd b/hippotatd
index b08ce66..f3d88d6 100755
--- a/hippotatd
+++ b/hippotatd
@@ -104,7 +104,8 @@ class Client():
 
   def _req_cancel(self, request):
     self._log(DBG.HTTP_CTRL, 'cancel', idof=request)
-    request.finish()
+    try: request.finish()
+    except Exception: pass
 
   def _req_error(self, err, request):
     self._log(DBG.HTTP_CTRL, 'error %s' % err, idof=request)