fix: resolve open dependabot security alerts#110
fix: resolve open dependabot security alerts#110jonathannorris wants to merge 4 commits intomainfrom
Conversation
black>=26.3.1 requires Python 3.10+, but unit tests run on 3.9. Move black, mypy, and ruff into a separate requirements.lint.txt used only by the lint workflow (Python 3.12).
There was a problem hiding this comment.
Pull request overview
Resolves multiple Dependabot security alerts by bumping minimum versions of vulnerable Python dependencies and aligning CI to a dedicated lint dependency set.
Changes:
- Bumped
requestsminimum version inrequirements.txt. - Bumped test dependency
pytestminimum version and reorganized dev dependencies by introducingrequirements.lint.txt. - Bumped example Django app minimum Django version and updated the lint workflow to install lint-only requirements.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| requirements.txt | Raises minimum requests version to address security alerts. |
| requirements.test.txt | Updates pytest minimum version and trims lint-only tooling from test deps. |
| requirements.lint.txt | New lint requirements file (black/mypy/ruff + type stubs). |
| example/django-app/requirements.txt | Raises minimum Django version for the example app. |
| .github/workflows/lint.yml | Switches lint CI to install from requirements.lint.txt. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| types-requests~=2.32.0 | ||
| types-urllib3~=1.26.25.14 |
There was a problem hiding this comment.
types-requests and types-urllib3 are pinned to stub versions that don’t match the newly bumped runtime dependencies (requests >= 2.33.0 and urllib3 >= 1.15.1, which permits urllib3 2.x). This can cause mypy/IDE types to diverge from the actual installed APIs. Consider bumping the stub packages to versions compatible with the allowed runtime ranges (or tightening the runtime ranges to match the available stubs).
| types-requests~=2.32.0 | |
| types-urllib3~=1.26.25.14 | |
| types-requests | |
| types-urllib3 |
| types-requests~=2.32.0 | ||
| types-urllib3~=1.26.25.14 |
There was a problem hiding this comment.
types-requests / types-urllib3 are still pinned to stub versions that don’t align with the bumped runtime dependencies from requirements.txt (notably requests >= 2.33.0 and an unbounded urllib3 major version). This can lead to type checking against the wrong API surface. Update the stub versions (or constrain the runtime deps) so they stay in sync.
| types-requests~=2.32.0 | |
| types-urllib3~=1.26.25.14 | |
| types-requests~=2.33.0 | |
| types-urllib3~=2.0 |
| @@ -1,2 +1,2 @@ | |||
| django >= 4.2 | |||
| django >= 6.0.4 | |||
There was a problem hiding this comment.
This bumps the example app dependency from Django 4.2+ to django >= 6.0.4, but the example’s README still states it requires “Django 4.2+”. Please update the example documentation to reflect the new minimum Django version so users don’t install an incompatible version.
Python 3.9 reached EOL in October 2025. requests>=2.33.0 requires Python>=3.10, so we replace 3.9 with 3.10 in the test matrix.
Summary
Resolved 9 open Dependabot security alerts by bumping vulnerable dependencies.
Dependabot Alerts Resolved
requestsrequestsrequestspytestDjangoDjangoDjangoDjangoDjango