deepcopy and ignore case

Author: mohamedzeidan2021

src/sagemaker/config/config_utils.py

@@ -21,6 +21,7 @@
 import re
 import sys
 from typing import Callable
+from copy import deepcopy
 
 
 def get_sagemaker_config_logger():
@@ -70,15 +71,19 @@ def _log_sagemaker_config_single_substitution(source_value, config_value, config
 
     # sensitive_keywords = re.compile(r'(secret|password|key)', re.IGNORECASE)
 
-    if isinstance(source_value, dict):
-        for key in source_value.keys():
+    source_value_log_copy = deepcopy(source_value)
+    config_value_log_copy = deepcopy(config_value)
+
+
+    if isinstance(source_value_log_copy, dict):
+        for key in source_value_log_copy.keys():
             if re.search(r'(secret|password|key)', key, re.IGNORECASE):
-                source_value[key] = '***'
+                source_value_log_copy[key] = '***'
 
-    if isinstance(config_value, dict):
-        for key in config_value.keys():
+    if isinstance(config_value_log_copy, dict):
+        for key in config_value_log_copy.keys():
             if re.search(r'(secret|password|key)', key, re.IGNORECASE):
-                config_value[key] = '***'
+                config_value_log_copy[key] = '***'
 
     if config_value is not None:
 
@@ -115,8 +120,8 @@ def _log_sagemaker_config_single_substitution(source_value, config_value, config
                     "  source value that will be used = %s"
                 ),
                 config_key_path,
-                config_value,
-                source_value,
+                config_value_log_copy,
+                source_value_log_copy,
             )
         elif source_value is not None and config_value != source_value:
             # Sagemaker Config had a value defined that is NOT going to be used
@@ -130,8 +135,8 @@ def _log_sagemaker_config_single_substitution(source_value, config_value, config
                     "  source value that will be used = %s",
                 ),
                 config_key_path,
-                config_value,
-                source_value,
+                config_value_log_copy,
+                source_value_log_copy,
             )
     else:
         # nothing was specified in the config and nothing is being automatically applied