Merge pull request #1875 from Kriechi/dont-log-tracebacks

don't log full tracebacks

mitmproxy/proxy/protocol/http2.py

@@ -1,6 +1,5 @@
 import threading
 import time
-import traceback
 import functools
 from typing import Dict, Callable, Any, List  # noqa
 
@@ -358,7 +357,6 @@ class Http2Layer(base.Layer):
                     self._cleanup_streams()
         except Exception as e:  # pragma: no cover
             self.log(repr(e), "info")
-            self.log(traceback.format_exc(), "debug")
             self._kill_all_streams()
 
 
@@ -624,7 +622,6 @@ class Http2SingleStreamLayer(httpbase._HttpTransmissionLayer, basethread.BaseThr
             pass
         except exceptions.ProtocolException as e:  # pragma: no cover
             self.log(repr(e), "info")
-            self.log(traceback.format_exc(), "debug")
         except exceptions.SetServerNotAllowedException as e:  # pragma: no cover
             self.log("Changing the Host server for HTTP/2 connections not allowed: {}".format(e), "info")
         except exceptions.Kill:

mitmproxy/proxy/protocol/http_replay.py

@@ -1,5 +1,3 @@
-import traceback
-
 from mitmproxy import log
 from mitmproxy import controller
 from mitmproxy import exceptions
@@ -107,10 +105,10 @@ class RequestReplayThread(basethread.BaseThread):
                 "log",
                 log.LogEntry("Connection killed", "info")
             )
-        except Exception:
+        except Exception as e:
             self.channel.tell(
                 "log",
-                log.LogEntry(traceback.format_exc(), "error")
+                log.LogEntry(repr(e), "error")
             )
         finally:
             r.first_line_format = first_line_format_backup

mitmproxy/proxy/server.py

@@ -120,7 +120,6 @@ class ConnectionHandler:
         except exceptions.Kill:
             self.log("Connection killed", "info")
         except exceptions.ProtocolException as e:
-
             if isinstance(e, exceptions.ClientHandshakeException):
                 self.log(
                     "Client Handshake failed. "
@@ -134,7 +133,7 @@ class ConnectionHandler:
             else:
                 self.log(str(e), "warn")
 
-                self.log(traceback.format_exc(), "debug")
+                self.log(repr(e), "debug")
             # If an error propagates to the topmost level,
             # we send an HTTP error response, which is both
             # understandable by HTTP clients and humans.