-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
vdo: Use yaml.safe_load() instead of yaml.load() #5632
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for your contribution! Can you please add a changelog fragment? Thanks.
yaml.load() without specifying a Loader= is deprecated and unsafe. For details, see https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation
changelogs/fragments/5632-vdo-Use-yaml-safe-load-instead-of-yaml-load.yml
Outdated
Show resolved
Hide resolved
…ml-load.yml Co-authored-by: Felix Fontein <felix@fontein.de>
Done, and also committed your suggestion. Thanks for the quick response! |
Backport to stable-4: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 428e181 on top of patchback/backports/stable-4/428e181440dc92378c7326d9d2bfadda052ce372/pr-5632 Backporting merged PR #5632 into main
🤖 @patchback |
Backport to stable-5: 💚 backport PR created✅ Backport PR branch: Backported as #5636 🤖 @patchback |
* vdo: Use yaml.safe_load() instead of yaml.load() yaml.load() without specifying a Loader= is deprecated and unsafe. For details, see https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation * Update changelogs/fragments/5632-vdo-Use-yaml-safe-load-instead-of-yaml-load.yml Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Lee Garrett <lgarrett@rocketjump.eu> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit 428e181)
Backport to stable-6: 💚 backport PR created✅ Backport PR branch: Backported as #5637 🤖 @patchback |
@leegarrett thanks a lot again for reporting and fixing this! :) |
* vdo: Use yaml.safe_load() instead of yaml.load() yaml.load() without specifying a Loader= is deprecated and unsafe. For details, see https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation * Update changelogs/fragments/5632-vdo-Use-yaml-safe-load-instead-of-yaml-load.yml Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Lee Garrett <lgarrett@rocketjump.eu> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit 428e181)
* vdo: Use yaml.safe_load() instead of yaml.load() yaml.load() without specifying a Loader= is deprecated and unsafe. For details, see https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation * Update changelogs/fragments/5632-vdo-Use-yaml-safe-load-instead-of-yaml-load.yml Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Lee Garrett <lgarrett@rocketjump.eu> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit 428e181) Co-authored-by: Lee Garrett <leegarrett@users.noreply.github.com>
* vdo: Use yaml.safe_load() instead of yaml.load() yaml.load() without specifying a Loader= is deprecated and unsafe. For details, see https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation * Update changelogs/fragments/5632-vdo-Use-yaml-safe-load-instead-of-yaml-load.yml Co-authored-by: Felix Fontein <felix@fontein.de> Co-authored-by: Lee Garrett <lgarrett@rocketjump.eu> Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit 428e181) Co-authored-by: Lee Garrett <leegarrett@users.noreply.github.com>
yaml.load() without specifying a Loader= is deprecated and unsafe.
For details, see
https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation
SUMMARY
Fixes #5631
ISSUE TYPE
COMPONENT NAME
vdo