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
|
Keep this PR in a mergeable state → Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
|
Important Review skippedIgnore keyword(s) in the title. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
There was a problem hiding this comment.
Pull request overview
This PR updates Django from 2.2.28 to 4.2.27 and sqlparse from 0.4.4 to 0.5.4 to address 3 security vulnerabilities identified by Snyk. While the Django 4.2 version is supported by the project (as evidenced by the CHANGELOG and test configuration), the update creates an inconsistency with the source requirements file.
- Django upgraded from 2.2.28 to 4.2.27 (major version jump to address security vulnerabilities)
- sqlparse upgraded from 0.4.4 to 0.5.4 (transitive dependency update for Django compatibility)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # boto3 | ||
| # s3transfer | ||
| django==2.2.28 | ||
| django==4.2.27 |
There was a problem hiding this comment.
The updated Django version (4.2.27) in this compiled requirements file conflicts with the source constraint in requirements/django.in which specifies Django>=2.2,<3.0. This mismatch will cause issues when requirements are regenerated using make upgrade or pip-compile.
To fix this, you need to also update requirements/django.in to allow Django 4.x:
Django>=4.2,<4.3
This change is supported by the project - Django 4.2 support was added in version 1.7.0 (see CHANGELOG.rst line 25) and the tox.ini configuration includes Django 4.2 test environments.



Snyk has created this PR to fix 3 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):
requirements/django.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