Skip to content
This repository has been archived by the owner on Jan 7, 2024. It is now read-only.

Update pyyaml when 4.2 release is available due to CVE-2017-18342 #52

Closed
redshiftzero opened this issue Jan 11, 2019 · 0 comments · Fixed by #72
Closed

Update pyyaml when 4.2 release is available due to CVE-2017-18342 #52

redshiftzero opened this issue Jan 11, 2019 · 0 comments · Fixed by #72
Assignees

Comments

@redshiftzero
Copy link
Contributor

Description

We got a GitHub security alert regarding a CVE in pyyaml: CVE-2017-18342. The fix, merged in the pyyaml repo, renames safe_load to load and load to danger_load. pyyaml is brought in as a development only dependency for this project.

We shouldn't update to latest yet due to the fact that the only 4.2* versions available on PyPI are marked as pre-release - see yaml/pyyaml#193 for the full details.

tl;dr We should wait until pyyaml 4.2 is officially released and then update.

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

Successfully merging a pull request may close this issue.

2 participants