Suppress YAMLLoadWarning (#728)

* Explicitly specify yaml.load() to use Loader=FullLoader to suppress deprecated warning. * Use safe_load instead of FullLoad.
juju · Jul 22, 2022 · 96d51ee · 96d51ee
1 parent 8939617
commit 96d51ee
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/charmhelpers/contrib/templating/contexts.py b/charmhelpers/contrib/templating/contexts.py
@@ -118,7 +118,7 @@ def juju_state_to_yaml(yaml_path, namespace_separator=':',
 
     if os.path.exists(yaml_path):
         with open(yaml_path, "r") as existing_vars_file:
-            existing_vars = yaml.load(existing_vars_file.read())
+            existing_vars = yaml.safe_load(existing_vars_file.read())
     else:
         with open(yaml_path, "w+"):
             pass

diff --git a/tools/charm_helpers_sync/charm_helpers_sync.py b/tools/charm_helpers_sync/charm_helpers_sync.py
@@ -34,7 +34,7 @@ def parse_config(conf_file):
     if not os.path.isfile(conf_file):
         logging.error('Invalid config file: %s.' % conf_file)
         return False
-    return yaml.load(open(conf_file).read())
+    return yaml.safe_load(open(conf_file).read())
 
 
 def clone_helpers(work_dir, repo):