diff options
| author | Yehiyam Livneh <yehiyam@gmail.com> | 2019-11-24 17:14:00 +0200 |
|---|---|---|
| committer | Yehiyam Livneh <yehiyam@gmail.com> | 2019-11-24 17:14:00 +0200 |
| commit | 2f85766642fab2d5c0994429c8506afeafa24fa9 (patch) | |
| tree | 4b8f9da10f4adac33222b0601bdde9152e157408 | |
| parent | 3c25814664fef5b78716ed8841123ed1c0d17824 (diff) | |
| download | websocket-client-2f85766642fab2d5c0994429c8506afeafa24fa9.tar.gz | |
avoid calling repr(data) if tracing is not enabled
| -rw-r--r-- | .gitignore | 2 | ||||
| -rw-r--r-- | websocket/_core.py | 3 | ||||
| -rw-r--r-- | websocket/_logging.py | 6 |
3 files changed, 8 insertions, 3 deletions
@@ -9,3 +9,5 @@ dist .cache/ compliance/reports/ +.eggs +.vscode
\ No newline at end of file diff --git a/websocket/_core.py b/websocket/_core.py index 0f914c2..418aafc 100644 --- a/websocket/_core.py +++ b/websocket/_core.py @@ -271,7 +271,8 @@ class WebSocket(object): frame.get_mask_key = self.get_mask_key data = frame.format() length = len(data) - trace("send: " + repr(data)) + if (isEnabledForTrace()): + trace("send: " + repr(data)) with self.lock: while data: diff --git a/websocket/_logging.py b/websocket/_logging.py index 70a6271..c947778 100644 --- a/websocket/_logging.py +++ b/websocket/_logging.py @@ -34,7 +34,7 @@ _logger.addHandler(NullHandler()) _traceEnabled = False __all__ = ["enableTrace", "dump", "error", "warning", "debug", "trace", - "isEnabledForError", "isEnabledForDebug"] + "isEnabledForError", "isEnabledForDebug", "isEnabledForTrace"] def enableTrace(traceable, handler = logging.StreamHandler()): @@ -49,7 +49,6 @@ def enableTrace(traceable, handler = logging.StreamHandler()): _logger.addHandler(handler) _logger.setLevel(logging.DEBUG) - def dump(title, message): if _traceEnabled: _logger.debug("--- " + title + " ---") @@ -80,3 +79,6 @@ def isEnabledForError(): def isEnabledForDebug(): return _logger.isEnabledFor(logging.DEBUG) + +def isEnabledForTrace(): + return _traceEnabled |