diff options
| author | liris <liris.pp@gmail.com> | 2019-12-25 21:09:28 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-12-25 21:09:28 +0900 |
| commit | 562808fe25f4f922e3422b99eb00aa53d9734ba0 (patch) | |
| tree | 9834079e9edab008239dd78cee54b56048ca915f /websocket | |
| parent | 647820961d44f56591d5e5a1c42888150a57bf48 (diff) | |
| parent | 2f85766642fab2d5c0994429c8506afeafa24fa9 (diff) | |
| download | websocket-client-562808fe25f4f922e3422b99eb00aa53d9734ba0.tar.gz | |
Merge pull request #584 from kube-HPC/add-condition-for-trace
avoid calling repr(data) if tracing is not enabled
Diffstat (limited to 'websocket')
| -rw-r--r-- | websocket/_core.py | 3 | ||||
| -rw-r--r-- | websocket/_logging.py | 6 |
2 files changed, 6 insertions, 3 deletions
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 |