Skip to content
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

YAMLLoadWarning, potentially unsafe code #215

Closed
0anton opened this issue Apr 13, 2019 · 2 comments
Closed

YAMLLoadWarning, potentially unsafe code #215

0anton opened this issue Apr 13, 2019 · 2 comments
Milestone

Comments

@0anton
Copy link

0anton commented Apr 13, 2019

I am getting tons of warnings when I am running:

jupytext --to notebook xx.py

(version - 1.0.5):

/opt/conda/lib/python3.7/site-packages/jupytext/header.py:174: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  metadata.update(yaml.load('\n'.join(jupyter))['jupyter'])

Should not we been using

yaml.safe_load()

instead of

yaml.load()

?

@mwouts
Copy link
Owner

mwouts commented Apr 14, 2019

Hello @0anton , thanks for reporting this. We have included the corresponding change in version 1.1. Do you want to give a try to the latest release?

pip install jupytext==1.1.0

@mwouts mwouts added this to the 1.1.0 milestone Apr 14, 2019
@mwouts mwouts closed this as completed Apr 14, 2019
@0anton
Copy link
Author

0anton commented Apr 16, 2019

Hi Marc!
Yes, 1.1.0 does not writes warning anymore! Thank you for your prompt reply!
Kind regards,
Anton

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

No branches or pull requests

2 participants