Skip to content

Commit 31ade5e

Browse files
committed
jadepy: avoid expensive message formatting for lines that won't be logged
1 parent c27d651 commit 31ade5e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

jadepy/jade.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2200,7 +2200,8 @@ def serialise_cbor_request(request):
22002200
"""
22012201
dump = cbor.dumps(request)
22022202
logger.info(f'Sending {request["method"]} request {request["id"]} length {len(dump)}')
2203-
logger.debug(f'Sending: {_hexlify(request)}')
2203+
if logger.isEnabledFor(logging.DEBUG):
2204+
logger.debug(f'Sending: {_hexlify(request)}')
22042205
return dump
22052206

22062207
def write(self, bytes_):
@@ -2271,7 +2272,8 @@ def read_cbor_message(self):
22712272
# A message response (to a prior request)
22722273
if 'id' in message:
22732274
logger.info(f'Received reply {message["id"]}')
2274-
logger.debug(f'Received: {_hexlify(message)}')
2275+
if logger.isEnabledFor(logging.DEBUG):
2276+
logger.debug(f'Received: {_hexlify(message)}')
22752277
return message
22762278

22772279
# A log message - handle as normal

0 commit comments

Comments
 (0)