fix(consumer): Some log lines were not logging the provided data.

Author: priiiiit

ct_configrecorder_override_consumer.py

@@ -25,6 +25,7 @@
 import os
 
 
+
 def lambda_handler(event, context):
     LOG_LEVEL = os.getenv('LOG_LEVEL')
     logging.getLogger().setLevel(LOG_LEVEL)
@@ -80,7 +81,7 @@ def assume_role(account_id, role='AWSControlTowerExecution'):
 
         # Describe configuration recorder
         configrecorder = configservice.describe_configuration_recorders()
-        logging.info(f'Existing Configuration Recorder :', configrecorder)
+        logging.info(f'Existing Configuration Recorder: {configrecorder}')
 
         # ControlTower created configuration recorder with name "aws-controltower-BaselineConfigRecorder" and we will update just that
         try:
@@ -100,6 +101,9 @@ def assume_role(account_id, role='AWSControlTowerExecution'):
 
             # Event = Delete is when stack is deleted, we rollback changed made and leave it as ControlTower Intended
             if event == 'Delete':
+                logging.warn(
+                    "Deleting stack: Reverting Configuration Recorder to default settings"
+                )
                 response = configservice.put_configuration_recorder(
                     ConfigurationRecorder={
                         'name': 'aws-controltower-BaselineConfigRecorder',
@@ -110,7 +114,9 @@ def assume_role(account_id, role='AWSControlTowerExecution'):
                         },
                         'recordingMode': {'recordingFrequency': 'CONTINUOUS'},
                     })
-                logging.info(f'Response for put_configuration_recorder :{response} ')
+                logging.info(
+                    f"Configuration Recorder reset to default. Response: {json.dumps(response, default=str)}"
+                )
 
             else:
                 config_recorder = {
@@ -152,7 +158,7 @@ def assume_role(account_id, role='AWSControlTowerExecution'):
             logging.info(f'Post Change Configuration recorder : {configrecorder}')
 
         except botocore.exceptions.ClientError as exe:
-            logging.error('Unable to Update Config Recorder for Account and Region : ', account_id, aws_region)
+            logging.error(f'Unable to Update Config Recorder for Account {account_id} and Region {aws_region}')
             configrecorder = configservice.describe_configuration_recorders()
             logging.info(f'Exception : {configrecorder}')
             raise exe