Skip to content

[Snyk] Fix for 2 vulnerabilities #495

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.

Sign up for GitHub

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

Open
robertatakenaka wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[Snyk] Fix for 2 vulnerabilities #495

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion requirements/local.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
-r base.txt

Werkzeug==2.2.3 # https://github.com/pallets/werkzeug
Werkzeug==3.1.5 # https://github.com/pallets/werkzeug

Copilot AI Jan 12, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upgrading Werkzeug from 2.2.3 to 3.1.5 is a major version upgrade that may contain breaking changes. Werkzeug 3.0 introduced several breaking changes including removal of deprecated APIs and changes to request/response handling. While no direct usage of Werkzeug was found in the codebase, this is a transitive dependency likely used by Flask or other web frameworks. Ensure that all frameworks and libraries depending on Werkzeug are compatible with version 3.x, and verify that tests pass before merging.

Suggested change
Werkzeug==3.1.5 # https://github.com/pallets/werkzeug
Werkzeug==2.2.3 # https://github.com/pallets/werkzeug

Copilot uses AI. Check for mistakes.
ipdb==0.13.13 # https://github.com/gotcha/ipdb
psycopg2==2.9.3 # https://github.com/psycopg/psycopg2
watchgod==0.7 # https://github.com/samuelcolvin/watchgod
Expand Down Expand Up @@ -35,3 +35,4 @@ django-debug-toolbar # https://github.com/jazzband/django-debug-toolbar
django-extensions==3.2.1 # https://github.com/django-extensions/django-extensions
django-coverage-plugin==2.0.2 # https://github.com/nedbat/django_coverage_plugin
pytest-django==4.5.2 # https://github.com/pytest-dev/pytest-django
urllib3>=2.6.3 # not directly required, pinned by Snyk to avoid a vulnerability

Copilot AI Jan 12, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The urllib3 security fix is only applied to local.txt (development environment) but not to production.txt. If urllib3 is a transitive dependency used in production, the production environment may remain vulnerable. Consider adding the same urllib3 pinning to requirements/production.txt to ensure the security fix is applied across all environments.

Copilot uses AI. Check for mistakes.

Copilot AI Jan 12, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The version specification for urllib3 uses ">=" while all other dependencies in this file use "==". This is inconsistent with the project's dependency pinning pattern. Consider using "urllib3==2.6.3" instead to ensure reproducible builds and prevent unexpected behavior from future patch releases.

Suggested change
urllib3>=2.6.3 # not directly required, pinned by Snyk to avoid a vulnerability
urllib3==2.6.3 # not directly required, pinned by Snyk to avoid a vulnerability

Copilot uses AI. Check for mistakes.