Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
2 changes: 1 addition & 1 deletion python/cloud-run-django-hello-world/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
django==4.2.10
requests==2.31.0
requests==2.32.2
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The requests dependency is imported in helloapp/views.py but not utilized in any of the view functions. If it's not required for the sample, consider removing both the unused import and this dependency to keep the project clean and reduce the security surface area.

debugpy # Required for debugging.
2 changes: 1 addition & 1 deletion python/cloud-run-python-hello-world/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Flask==2.3.3
requests==2.31.0
requests==2.32.2
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

While updating requests to version 2.32.2 is important for security (fixing CVE-2024-35195), this dependency appears to be unused in the current sample. It is not imported or used in app.py. If it is not required for the sample's functionality, consider removing it to reduce the project's dependency footprint and attack surface.

debugpy # Required for debugging.
2 changes: 1 addition & 1 deletion python/django/python-hello-world/src/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Django==4.2.10
requests==2.31.0
requests==2.32.2
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The requests dependency appears to be unused in this sample, as it is not imported in helloapp/views.py or helloapp/tests.py. Unless it is used in a part of the project not shown (such as settings.py), consider removing it to maintain a minimal set of dependencies and improve security.

psycopg2-binary==2.9.9
debugpy # Required for debugging
2 changes: 1 addition & 1 deletion python/python-guestbook/src/frontend/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Flask==2.3.3
requests==2.31.0
requests==2.32.2
python-dateutil==2.8.2
debugpy # Required for debugging