Use traceback.format_exc and log instead of traceback.print_exc()

supervisor/http.py

@@ -823,7 +823,8 @@ def make_http_servers(options, supervisord):
             try:
                 inst = factory(supervisord, **d)
             except:
-                import traceback; traceback.print_exc()
+                tb = traceback.format_exc()
+                options.logger.warn(tb)
                 raise ValueError('Could not make %s rpc interface' % name)
             subinterfaces.append((name, inst))
             options.logger.info('RPC interface %r initialized' % name)

supervisor/tests/test_xmlrpc.py

@@ -435,14 +435,15 @@ class TestDeferredXMLRPCResponse(unittest.TestCase):
             raise ValueError('foo')
         callback.delay = 1
         inst = self._makeOne(callback=callback)
-        inst.traceback = Dummy()
-        called = []
-        inst.traceback.print_exc = lambda: called.append(True)
         self.assertEqual(inst.more(), None)
         self.assertEqual(inst.request._error, 500)
         self.assertTrue(inst.finished)
-        self.assertTrue(called)
-        
+        logged = inst.request.channel.server.logger.logged
+        self.assertEqual(len(logged), 1)
+        src, msg = logged[0]
+        self.assertEqual(src, 'XML-RPC response callback error')
+        self.assertTrue("Traceback" in msg)
+
 class DummyResponse:
     def __init__(self, status=200, body='', reason='reason'):
         self.status = status

supervisor/xmlrpc.py

@@ -51,7 +51,6 @@ class DeferredXMLRPCResponse:
     """ A medusa producer that implements a deferred callback; requires
     a subclass of asynchat.async_chat that handles NOT_DONE_YET sentinel """
     CONNECTION = re.compile ('Connection: (.*)', re.IGNORECASE)
-    traceback = traceback # for testing override
 
     def __init__(self, request, callback):
         self.callback = callback
@@ -77,8 +76,10 @@ class DeferredXMLRPCResponse:
             return self.getresponse(body)
 
         except:
-            # report unexpected exception back to server
-            self.traceback.print_exc()
+            tb = traceback.format_exc()
+            self.request.channel.server.logger.log(
+                "XML-RPC response callback error", tb
+                )
             self.finished = True
             self.request.error(500)