-
Notifications
You must be signed in to change notification settings - Fork 10
[Snyk] Fix for 22 vulnerabilities #1261
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
base: main
Are you sure you want to change the base?
Conversation
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-10074036 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-10302884 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-12485156 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-13836728 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-13837025 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-14157807 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-14157810 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7435780 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7436273 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7436514 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7436646 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7642790 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7642791 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7642813 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7642814 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7886958 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-7886959 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-8456315 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-8456316 - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-9296408 - https://snyk.io/vuln/SNYK-PYTHON-SQLPARSE-14157217 - https://snyk.io/vuln/SNYK-PYTHON-SQLPARSE-6615674
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This Snyk PR aims to fix 22 security vulnerabilities (20 in Django and 2 in sqlparse) by upgrading Django from 3.2.25 to 4.2.27 and sqlparse from 0.4.4 to 0.5.4. However, there's a critical discrepancy: the actual changes only add a sqlparse version constraint, while the codebase already shows Django 5.2.7 installed (not 3.2.25 as the PR description suggests).
Changes:
- Adds sqlparse>=0.5.4 constraint to fix 2 vulnerabilities
- No Django upgrade is included despite PR claiming to fix 20 Django vulnerabilities
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # Xlwt | ||
| # ------------------------------------------------------------------------------ | ||
| xlwt==1.3.0 | ||
| sqlparse>=0.5.4 # not directly required, pinned by Snyk to avoid a vulnerability |
Copilot
AI
Jan 25, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR description includes warnings about dependency incompatibilities that this change may introduce:
- "flower 2.0.1 requires tornado, which is not installed"
- "django-rosetta 0.10.1 has requirement Django>=4.2, but you have Django 3.2.25"
- "django-prometheus 2.4.1 has requirement Django<6.0,>=4.2, but you have Django 3.2.25"
- "celery 5.3.0 requires kombu, which is not installed"
While the warnings reference Django 3.2.25 (suggesting the analysis may be outdated since the file shows Django 5.2.7), the missing dependencies (tornado and kombu) could cause runtime issues. Please verify that:
- All required transitive dependencies are properly installed
- The dependency warnings are accurate for the current state of the project
- The project will function correctly after this change
| # Xlwt | ||
| # ------------------------------------------------------------------------------ | ||
| xlwt==1.3.0 | ||
| sqlparse>=0.5.4 # not directly required, pinned by Snyk to avoid a vulnerability |
Copilot
AI
Jan 25, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR description states this change upgrades Django from 3.2.25 to 4.2.27 to fix 22 vulnerabilities (20 in Django, 2 in sqlparse). However, the actual diff only adds a sqlparse version constraint and does not include any Django upgrade. The requirements file currently shows Django 5.2.7 at line 19, which contradicts the PR description's claim of upgrading from Django 3.2.25.
This discrepancy suggests either:
- The PR description is incorrect or outdated
- The Django upgrade was already applied separately
- There's a mismatch between what Snyk detected and the actual codebase state
Please verify that the correct changes are being made and update either the PR description or the changes accordingly.
Snyk has created this PR to fix 22 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):
requirements/base.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:
🦉 Allocation of Resources Without Limits or Throttling
🦉 Improper Output Neutralization for Logs
🦉 SQL Injection
🦉 More lessons are available in Snyk Learn