[Snyk] Security upgrade django from 3.2.25 to 4.2.28#1315
[Snyk] Security upgrade django from 3.2.25 to 4.2.28#1315robertatakenaka wants to merge 2 commits intomainfrom
Conversation
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-15183335
There was a problem hiding this comment.
Pull request overview
Updates local development requirements in response to a Snyk-reported Django vulnerability, aiming to ensure an upgraded Django version is installed when using requirements/local.txt.
Changes:
- Normalizes the
django-silkrequirement line formatting. - Adds a new
django>=4.2.28requirement inrequirements/local.txt.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| django-silk==5.3.2 | ||
| django>=4.2.28 # not directly required, pinned by Snyk to avoid a vulnerability No newline at end of file |
There was a problem hiding this comment.
requirements/local.txt includes -r base.txt, and requirements/base.txt already pins django==5.2.7 (base.txt:19). Adding django>=4.2.28 here is redundant and introduces a second (less specific) constraint, which also doesn’t match the PR title/description claiming an upgrade from 3.2.25. Suggest removing this line and, if a Django security bump is needed, adjusting the pinned version in requirements/base.txt instead (or ensure Snyk is scanning the right requirements file).
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-DJANGO-15183335
Snyk has created this PR to fix 1 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