Skip to content

fix: upgrade requests and add workflow permissions#19

Merged
abhishek-anand merged 1 commit intomainfrom
fix/security-requests-and-workflow-permissions
Apr 8, 2026
Merged

fix: upgrade requests and add workflow permissions#19
abhishek-anand merged 1 commit intomainfrom
fix/security-requests-and-workflow-permissions

Conversation

@mkagenius
Copy link
Copy Markdown
Collaborator

@mkagenius mkagenius commented Apr 8, 2026

Summary

  • Upgrades requests from 2.32.4 to >=2.33.0 to resolve a known CVE (flagged by CodeQL)
  • Adds explicit permissions: contents: read to the Docker build workflow to address the missing permissions warning

Test plan

  • Verify Docker build workflow runs successfully
  • Confirm no dependency conflicts with requests>=2.33.0

🤖 Generated with Claude Code

- Upgrade requests from 2.32.4 to >=2.33.0 to fix CVE vulnerability
- Add explicit permissions block to docker-build workflow (contents: read)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mkagenius mkagenius self-assigned this Apr 8, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the requests library version in requirements.txt to address a security vulnerability. However, the specified version 2.33.0 (as well as the previous 2.32.4) does not exist on PyPI, which will cause the build process to fail. It is recommended to use a valid stable release such as 2.32.3.

@mkagenius mkagenius requested a review from abhishek-anand April 8, 2026 11:59
@abhishek-anand abhishek-anand merged commit a4a05a6 into main Apr 8, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants