From 207aae2b047b788e30d110ca588a7453b23d881f Mon Sep 17 00:00:00 2001
From: Pablo Hoffman <pablo@pablohoffman.com>
Date: Thu, 26 Nov 2009 19:12:33 -0200
Subject: [PATCH] Fixed logging of "Spider closed" message in engine

---
 scrapy/core/engine.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/scrapy/core/engine.py b/scrapy/core/engine.py
index 11cb0e98e..1fdaf476f 100644
--- a/scrapy/core/engine.py
+++ b/scrapy/core/engine.py
@@ -312,7 +312,9 @@ class ExecutionEngine(object):
         if call and call.active():
             call.cancel()
         dfd = defer.maybeDeferred(spiders.close_spider, spider)
-        dfd.addBoth(log.msg, "Spider closed (%s)" % reason, spider=spider)
+        dfd.addErrback(log.err, "Unhandled error on SpiderManager.close_spider()",
+            spider=spider)
+        dfd.addBoth(lambda _: log.msg("Spider closed (%s)" % reason, spider=spider))
         reactor.callLater(0, self._mainloop)
         return dfd