Allow pyyaml version >4 in setup.py

[qboot]

hello!

I recently received this Github alert on a project because I'm using a pyyaml version <4.2b1:

CVE-2017-18342
high severity
Vulnerable versions: < 4.2b1
Patched version: 4.2b1
In PyYAML before 4.1, the yaml.load() API could execute arbitrary code. In other words, yaml.safe_load is not used.

After some research it seems that it's because docker-compose require PyYAML >= 3.10, < 4 in setup.py :

compose/setup.py

Line 35 in 64633a8

'PyYAML >= 3.10, < 4',

Is there any contraindication to not authorize version >4 ? Thanks.

[shin-]

Hi @qboot

The vulnerable code is not currently used in our code base (Compose only ever calls safe_load). That said, we'll definitely look into upgrading the dependency to avoid issues cropping up in the future.

[hakan458]

ERROR: docker-compose 1.24.0 has requirement PyYAML<4.3,>=3.10, but you'll have pyyaml 5.1 which is incompatible.

Any plans for this?

[mayankkgandhi]

ERROR: docker-compose 1.24.0 has requirement PyYAML<4.3,>=3.10, but you'll have pyyaml 5.1 which is incompatible.

Any plans for this?

I am having the same error for my docker image, did you find any solution yet?

[tannergilligan]

I'm also getting this error when using sagemaker, and it tries to run docker-compose:

pkg_resources.ContextualVersionConflict: (PyYAML 5.1 (c:\users\tanner\anaconda3\lib\site-packages), Requirement.parse('PyYAML<4.3,>=3.10'), {'docker-compose'})

Edit: I found this thread #6619

[rmwenzel]

I ran into the same issue trying to use sagemaker and keras. It can be resolved by using PyYAML==4.2b1 -- see swagger-atlas/atlas/issues/7