From 096176fe63f8728ebecf5b274f8b24fe4b4c5895 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Thu, 5 Mar 2026 04:32:11 -0600 Subject: [PATCH] fix: remove hardcoded master_debug flag from api_calls.py (#28) Remove master_debug = True and replace all conditional INFO/DEBUG log calls with unconditional logger.debug(). Also switches log_return_value to logger.debug and removes the associated dead commented-out code. Co-Authored-By: Claude Sonnet 4.6 --- api_calls.py | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/api_calls.py b/api_calls.py index bb969b4..7cf2b31 100644 --- a/api_calls.py +++ b/api_calls.py @@ -17,7 +17,6 @@ DB_URL = ( if "prod" in ENV_DATABASE else "https://pddev.manticorum.com/api" ) -master_debug = True PLAYER_CACHE = {} logger = logging.getLogger("discord_app") @@ -53,14 +52,10 @@ def log_return_value(log_string: str): line = log_string[start:end] if len(line) == 0: return - logger.info(f"{'\n\nreturn: ' if start == 0 else ''}{log_string[start:end]}") + logger.debug(f"{'\n\nreturn: ' if start == 0 else ''}{log_string[start:end]}") start += 3000 end += 3000 logger.warning("[ S N I P P E D ]") - # if master_debug: - # logger.info(f'return: {log_string[:1200]}{" [ S N I P P E D ]" if len(log_string) > 1200 else ""}\n') - # else: - # logger.debug(f'return: {log_string[:1200]}{" [ S N I P P E D ]" if len(log_string) > 1200 else ""}\n') async def db_get( @@ -93,7 +88,7 @@ async def db_get( """ req_url = get_req_url(endpoint, api_ver=api_ver, object_id=object_id, params=params) log_string = f"db_get - get: {endpoint} id: {object_id} params: {params}" - logger.info(log_string) if master_debug else logger.debug(log_string) + logger.debug(log_string) for attempt in range(retries): try: @@ -150,7 +145,7 @@ async def db_patch( """ req_url = get_req_url(endpoint, api_ver=api_ver, object_id=object_id, params=params) log_string = f"db_patch - patch: {endpoint} {params}" - logger.info(log_string) if master_debug else logger.debug(log_string) + logger.debug(log_string) try: client_timeout = ClientTimeout(total=timeout) @@ -192,7 +187,7 @@ async def db_post( """ req_url = get_req_url(endpoint, api_ver=api_ver) log_string = f"db_post - post: {endpoint} payload: {payload}\ntype: {type(payload)}" - logger.info(log_string) if master_debug else logger.debug(log_string) + logger.debug(log_string) try: client_timeout = ClientTimeout(total=timeout) @@ -234,7 +229,7 @@ async def db_put( """ req_url = get_req_url(endpoint, api_ver=api_ver) log_string = f"db_put - put: {endpoint} payload: {payload}\ntype: {type(payload)}" - logger.info(log_string) if master_debug else logger.debug(log_string) + logger.debug(log_string) try: client_timeout = ClientTimeout(total=timeout) @@ -274,7 +269,7 @@ async def db_delete(endpoint: str, object_id: int, api_ver: int = 2, timeout: in """ req_url = get_req_url(endpoint, api_ver=api_ver, object_id=object_id) log_string = f"db_delete - delete: {endpoint} {object_id}" - logger.info(log_string) if master_debug else logger.debug(log_string) + logger.debug(log_string) try: client_timeout = ClientTimeout(total=timeout)