From 6050ff16d4d6c6503d0b13bb8f34f1ec451b6034 Mon Sep 17 00:00:00 2001 From: Jakub Onderka Date: Sun, 25 Oct 2020 17:15:21 +0100 Subject: [PATCH] fix: Remove duplicate check if debug logging is enabled --- pymisp/api.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/pymisp/api.py b/pymisp/api.py index 078be9f..f18b777 100644 --- a/pymisp/api.py +++ b/pymisp/api.py @@ -2309,15 +2309,13 @@ class PyMISP: try: response_json = response.json() - if logger.isEnabledFor(logging.DEBUG): - logger.debug(response_json) + logger.debug(response_json) if isinstance(response_json, dict) and response_json.get('response') is not None: # Cleanup. response_json = response_json['response'] return response_json except Exception: - if logger.isEnabledFor(logging.DEBUG): - logger.debug(response.text) + logger.debug(response.text) if expect_json: raise PyMISPUnexpectedResponse(f'Unexpected response from server: {response.text}') if lenient_response_type and not response.headers['Accept'].startswith('application/json'): @@ -2365,8 +2363,7 @@ class PyMISP: 'Accept': f'application/{output_type}', 'content-type': 'application/json', 'User-Agent': user_agent}) - if logger.isEnabledFor(logging.DEBUG): - logger.debug(prepped.headers) + logger.debug(prepped.headers) settings = s.merge_environment_settings(req.url, proxies=self.proxies or {}, stream=None, verify=self.ssl, cert=self.cert) return s.send(prepped, timeout=self.timeout, **settings)