fix: remove hardcoded master_debug flag from api_calls.py (#28)
All checks were successful
Build Docker Image / build (push) Successful in 1m20s
Build Docker Image / build (pull_request) Successful in 46s

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 <noreply@anthropic.com>
This commit is contained in:
Cal Corum 2026-03-05 04:32:11 -06:00 committed by cal
parent d7af529763
commit 096176fe63

View File

@ -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)