update: improve error logging for CMAB decision fetch failures

Author: FarhanAnjum-opti

optimizely/decision_service.py

@@ -185,11 +185,10 @@ def _get_decision_for_cmab_experiment(
             }
         except Exception as e:
             error_message = Errors.CMAB_FETCH_FAILED_DETAILED.format(
-                experiment.key,
-                str(e)
+                experiment.key
             )
             if self.logger:
-                self.logger.error(error_message)
+                self.logger.error(f'{error_message} {str(e)}')
             return {
                 "error": True,
                 "result": None,

optimizely/helpers/enums.py

@@ -127,9 +127,9 @@ class Errors:
     ODP_INVALID_DATA: Final = 'ODP data is not valid.'
     ODP_INVALID_ACTION: Final = 'ODP action is not valid (cannot be empty).'
     MISSING_SDK_KEY: Final = 'SDK key not provided/cannot be found in the datafile.'
-    CMAB_FETCH_FAILED: Final = 'CMAB decision fetch failed with status: {}'
-    INVALID_CMAB_FETCH_RESPONSE: Final = 'Invalid CMAB fetch response'
-    CMAB_FETCH_FAILED_DETAILED: Final = 'Failed to fetch CMAB decision for experiment key "{}" - {}'
+    CMAB_FETCH_FAILED: Final = 'CMAB decision fetch failed with status: {}.'
+    INVALID_CMAB_FETCH_RESPONSE: Final = 'Invalid CMAB fetch response.'
+    CMAB_FETCH_FAILED_DETAILED: Final = 'Failed to fetch CMAB data for experiment {}.'
 
 
 class ForcedDecisionLogs: