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

CLI v1.18.218 uses version(s) of PyYAML that has a known vulnerability #5893

Closed
smimani-godaddy opened this issue Jan 22, 2021 · 1 comment
Closed
Labels
closed-for-staleness dependencies This issue is a problem in a dependency. feature-request A feature should be added or improved.

Comments

@smimani-godaddy
Copy link

smimani-godaddy commented Jan 22, 2021

Confirm by changing [ ] to [x] below to ensure that it's a bug:

Describe the bug
A clear and concise description of what the bug is.
AWS cli requires PyYAML<5.4,>=3.10 but these versions have a known vulnerability. https://snyk.io/vuln/pip:pyyaml
SDK version number
latest
Platform/OS/Hardware/Device
What are you running the cli on?
macOS

To Reproduce (observed behavior)
Steps to reproduce the behavior

Expected behavior
A clear and concise description of what you expected to happen.
Use a version of PyYAML that is vulnerability free
Logs/output
awscli 1.18.218 has requirement PyYAML<5.4,>=3.10; python_version != "3.4", but you'll have pyyaml 5.4 which is incompatible.
Get full traceback and error logs by adding --debug to the command.

Additional context
Add any other context about the problem here.

@smimani-godaddy smimani-godaddy added the needs-triage This issue or PR still needs to be triaged. label Jan 22, 2021
@smimani-godaddy smimani-godaddy changed the title CLI v1.18.218 uses a version of PyYAML that has a known vulnerability CLI v1.18.218 uses version(s) of PyYAML that has a known vulnerability Jan 22, 2021
@nateprewitt
Copy link
Member

Hi @smimani-godaddy, I believe there's already a PR (#5887) open around this. The CLI only uses safe_load, so it's not impacted by the current CVEs in PyYAML. We're working to validate we can raise the allowed pin, allowing other software to use an updated version.

Users who upgrade to the CLI v2 will avoid these routine CVEs with PyYAML since the dependency has been dropped entirely.

@nateprewitt nateprewitt added dependencies This issue is a problem in a dependency. and removed needs-triage This issue or PR still needs to be triaged. labels Jan 22, 2021
@kdaily kdaily added the closing-soon This issue will automatically close in 4 days unless further comments are made. label Feb 1, 2021
@github-actions github-actions bot added closed-for-staleness and removed closing-soon This issue will automatically close in 4 days unless further comments are made. labels Feb 1, 2021
@github-actions github-actions bot closed this as completed Feb 1, 2021
@kdaily kdaily added the feature-request A feature should be added or improved. label Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-for-staleness dependencies This issue is a problem in a dependency. feature-request A feature should be added or improved.
Projects
None yet
Development

No branches or pull requests

3 participants