Conversation
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-14157807 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-14157810 - https://snyk.io/vuln/SNYK-PYTHON-SQLPARSE-14157217 - https://snyk.io/vuln/SNYK-PYTHON-URLLIB3-14192442 - https://snyk.io/vuln/SNYK-PYTHON-URLLIB3-14192443
There was a problem hiding this comment.
Pull request overview
This is an automated security fix PR from Snyk that addresses 5 vulnerabilities by upgrading three dependencies: Django (from 3.2.25 to 4.2.27), sqlparse (from 0.4.4 to 0.5.4), and urllib3 (from 2.0.7 to 2.6.0). The PR adds minimum version constraints for these packages to requirements/local.txt to ensure transitive dependencies meet security requirements.
Key changes:
- Adds
django>=4.2.27to fix Django vulnerabilities (SNYK-PYTHON-DJANGO-14157807, SNYK-PYTHON-DJANGO-14157810) - Adds
sqlparse>=0.5.4to fix sqlparse vulnerability (SNYK-PYTHON-SQLPARSE-14157217) - Adds
urllib3>=2.6.0to fix urllib3 vulnerabilities (SNYK-PYTHON-URLLIB3-14192442, SNYK-PYTHON-URLLIB3-14192443)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| django-silk==5.3.2 No newline at end of file | ||
| django-silk==5.3.2 | ||
| django>=4.2.27 # not directly required, pinned by Snyk to avoid a vulnerability |
There was a problem hiding this comment.
This Django version pin conflicts with the existing django==5.2.7 in requirements/base.txt (line 19). Since local.txt includes -r base.txt, pip will encounter a conflict between the exact pin (5.2.7) and the minimum version constraint (>=4.2.27).
While the constraint allows 5.2.7, having conflicting declarations can cause confusion and unexpected behavior. Consider either:
- Removing this line if base.txt already pins Django at a secure version
- Upgrading the version in base.txt to match the security requirements
- Using a version range that doesn't conflict (e.g., django>=5.2.7)
| django>=4.2.27 # not directly required, pinned by Snyk to avoid a vulnerability |
| @@ -40,4 +40,7 @@ tornado>=6.3.3 # not directly required, pinned by Snyk to avoid a vulnerability | |||
|
|
|||
| django-test-migrations==1.3.0 | |||
There was a problem hiding this comment.
django-test-migrations==1.3.0 is already declared on line 15 and line 41. This creates a duplicate dependency declaration in the same requirements file, which can cause confusion and potential issues during dependency resolution.
| django-test-migrations==1.3.0 |
Snyk has created this PR to fix 5 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):
requirements/local.txtImportant
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
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic
Learn how to fix vulnerabilities with free interactive lessons:
🦉 SQL Injection
🦉 Allocation of Resources Without Limits or Throttling