From d2cd4e08c57c7240f5e465838f4ec832300a0802 Mon Sep 17 00:00:00 2001 From: Alan Rominger Date: Fri, 22 Nov 2024 07:47:48 -0500 Subject: [PATCH] Do not check error state if null (#15655) --- awx/conf/settings.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/awx/conf/settings.py b/awx/conf/settings.py index dfe2e50602..fc48d55379 100644 --- a/awx/conf/settings.py +++ b/awx/conf/settings.py @@ -97,10 +97,13 @@ def _ctit_db_wrapper(trans_safe=False): except DatabaseError as e: if trans_safe: cause = e.__cause__ - if cause and hasattr(cause, 'sqlstate'): + sqlstate = getattr(cause, 'sqlstate', None) + if cause and sqlstate: sqlstate = cause.sqlstate sqlstate_str = psycopg.errors.lookup(sqlstate) logger.error('SQL Error state: {} - {}'.format(sqlstate, sqlstate_str)) + else: + logger.error(f'Error reading something related to database settings: {str(e)}.') else: logger.exception('Error modifying something related to database settings.') finally: