mckinsey / vizro

Vizro is a toolkit for creating modular data visualization applications.
https://vizro.readthedocs.io/en/stable/
Apache License 2.0
2.65k stars 136 forks source link

[Snyk] Security upgrade aiohttp from 3.8.6 to 3.9.2 #294

Closed maxschulz-COL closed 8 months ago

maxschulz-COL commented 8 months ago

This PR was automatically created by Snyk using the credentials of a real user.


Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.

#### Changes included in this PR - Changes to the following files to upgrade the vulnerable dependencies to a fixed version: - vizro-ai/snyk/requirements.txt
⚠️ Warning ``` openai 0.28.1 requires aiohttp, which is not installed. ```
#### Vulnerabilities that will be fixed ##### By pinning: Severity | Priority Score (*) | Issue | Upgrade | Breaking Change | Exploit Maturity :-------------------------:|-------------------------|:-------------------------|:-------------------------|:-------------------------|:------------------------- ![medium severity](https://res.cloudinary.com/snyk/image/upload/w_20,h_20/v1561977819/icon/m.png "medium severity") | **581/1000**
**Why?** Recently disclosed, Has a fix available, CVSS 5.9 | Improper Limitation of a Pathname to a Restricted Directory ('Path Traversal')
[SNYK-PYTHON-AIOHTTP-6209406](https://snyk.io/vuln/SNYK-PYTHON-AIOHTTP-6209406) | `aiohttp:`
`3.8.6 -> 3.9.2`
| No | No Known Exploit ![medium severity](https://res.cloudinary.com/snyk/image/upload/w_20,h_20/v1561977819/icon/m.png "medium severity") | **718/1000**
**Why?** Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 6.5 | HTTP Request Smuggling
[SNYK-PYTHON-AIOHTTP-6209407](https://snyk.io/vuln/SNYK-PYTHON-AIOHTTP-6209407) | `aiohttp:`
`3.8.6 -> 3.9.2`
| No | Proof of Concept (*) Note that the real score may have changed since the PR was raised. Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded. Check the changes in this PR to ensure they won't cause issues with your project. ------------ **Note:** *You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.* For more information: 🧐 [View latest project report](https://app.snyk.io/org/vizro/project/c5b16d47-7abb-426e-8e60-468beb0e8aba?utm_source=github&utm_medium=referral&page=fix-pr) 🛠 [Adjust project settings](https://app.snyk.io/org/vizro/project/c5b16d47-7abb-426e-8e60-468beb0e8aba?utm_source=github&utm_medium=referral&page=fix-pr/settings) 📚 [Read more about Snyk's upgrade and patch logic](https://support.snyk.io/hc/en-us/articles/360003891078-Snyk-patches-to-fix-vulnerabilities) [//]: # (snyk:metadata:{"prId":"81cf7bfe-00a2-4526-89ec-ccc76d0a2e71","prPublicId":"81cf7bfe-00a2-4526-89ec-ccc76d0a2e71","dependencies":[{"name":"aiohttp","from":"3.8.6","to":"3.9.2"}],"packageManager":"pip","projectPublicId":"c5b16d47-7abb-426e-8e60-468beb0e8aba","projectUrl":"https://app.snyk.io/org/vizro/project/c5b16d47-7abb-426e-8e60-468beb0e8aba?utm_source=github&utm_medium=referral&page=fix-pr","type":"auto","patch":[],"vulns":["SNYK-PYTHON-AIOHTTP-6209406","SNYK-PYTHON-AIOHTTP-6209407"],"upgrade":[],"isBreakingChange":false,"env":"prod","prType":"fix","templateVariants":["updated-fix-title","pr-warning-shown","priorityScore"],"priorityScoreList":[581,718],"remediationStrategy":"vuln"}) --- **Learn how to fix vulnerabilities with free interactive lessons:** 🦉 [Improper Limitation of a Pathname to a Restricted Directory ('Path Traversal')](https://learn.snyk.io/lesson/directory-traversal/?loc=fix-pr)
huong-li-nguyen commented 8 months ago

@Anna-Xiong - could you take a look whether this is fine or if we even need it?

Anna-Xiong commented 8 months ago

@Anna-Xiong - could you take a look whether this is fine or if we even need it?

looks like we don't even need this package, it is the dependency from openai and snyk pined this version. since it is marked as medium risk, we can add it