Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update yaml.load() to avoid warning #1041

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update yaml.load() to avoid warning #1041

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 20, 2020

The yaml.load() function call throws the following warning when Loader isn't specified:

"YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details."

The updated default loader avoids arbitrary code execution after issuing a warning, however the warning can be avoided with yaml.load(input, Loader=) or yaml.<full|safe|unsafe>_load(input).

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants