Conversation
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-TORNADO-14400977 - https://snyk.io/vuln/SNYK-PYTHON-TORNADO-14400978 - https://snyk.io/vuln/SNYK-PYTHON-TORNADO-14400979
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on January 13
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| PyPDF2>=2.0.0 | ||
| reportlab>=3.5.0 | ||
| streamlit | ||
| tornado>=6.5.3 # not directly required, pinned by Snyk to avoid a vulnerability |
There was a problem hiding this comment.
Bug: Inline comment breaks setuptools package installation
The new tornado requirement includes an inline comment (# not directly required...), but the read_requirements() function in setup.py only filters out lines starting with # - it doesn't strip inline comments. This causes the entire line including the comment to be passed to install_requires, which is not a valid PEP 508 specifier. While pip install -r requirements.txt handles inline comments correctly, installations via pip install . or from PyPI will fail because setuptools receives an invalid requirement string.
2 participants
Snyk has created this PR to fix 3 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):
requirements.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:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.
Note
Pins
tornado>=6.5.3inrequirements.txtto address security vulnerabilities.Written by Cursor Bugbot for commit 7c9c7b5. This will update automatically on new commits. Configure here.