Duke-GCB / D4S2

Web service to facilitate notification and transfer of projects in DukeDS
MIT License
0 stars 0 forks source link

pyyaml vulnerability #195

Closed johnbradley closed 5 years ago

johnbradley commented 5 years ago

https://nvd.nist.gov/vuln/detail/CVE-2017-18342

Details: https://github.com/yaml/pyyaml/issues/243#issuecomment-451540701

The suggested version 4.1 is a prerelease so not a good option currently.

According to the details link above using yaml.safe_load instead of yaml.load fixes this issue.

We don't appear to be using yaml directly in our code so we may be able to remove this requirement. The problem may still exist in libraries we are using.

johnbradley commented 5 years ago

Fixed by #200