-
Notifications
You must be signed in to change notification settings - Fork 0
[Snyk] Fix for 3 vulnerabilities #32
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.
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
base: main
Are you sure you want to change the base?
Conversation
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-TORCH-10332644 - https://snyk.io/vuln/SNYK-PYTHON-TORCH-10332645 - https://snyk.io/vuln/SNYK-PYTHON-WHEEL-15053866
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Walkthrough
This PR addresses security vulnerabilities identified by Snyk by pinning two indirect dependencies to specific minimum versions. The torch library is pinned to version 2.10.0 or higher, and the wheel package is pinned to version 0.46.2 or higher. These are transitive dependencies being explicitly constrained for security purposes rather than direct project requirements. The change also includes a minor formatting improvement by adding a newline at the end of the requirements file to comply with standard file formatting conventions.
Changes
| File(s) | Summary |
|---|---|
requirements.txt |
Pinned torch>=2.10.0 and wheel>=0.46.2 as indirect dependencies to address Snyk-identified security vulnerabilities; added trailing newline for proper file formatting. |
Sequence Diagram
This diagram shows the interactions between components:
sequenceDiagram
participant Dev as Developer
participant Snyk as Snyk Security Scanner
participant Deps as Dependency Manager
participant Repo as Repository
Dev->>Snyk: Run security scan
Snyk->>Snyk: Detect vulnerabilities in torch & wheel
Snyk-->>Dev: Report vulnerabilities
Dev->>Deps: Update requirements.txt
Note over Deps: Pin torch>=2.10.0<br/>Pin wheel>=0.46.2
Dev->>Repo: Commit dependency updates
Repo-->>Dev: Changes recorded
Install the extension
Note for Windsurf
Please change the default marketplace provider to the following in the windsurf settings:Marketplace Extension Gallery Service URL: https://marketplace.visualstudio.com/_apis/public/gallery
Marketplace Gallery Item URL: https://marketplace.visualstudio.com/items
Entelligence.ai can learn from your feedback. Simply add 👍 / 👎 emojis to teach it your preferences. More shortcuts below
Emoji Descriptions:
⚠️ Potential Issue - May require further investigation.- 🔒 Security Vulnerability - Fix to ensure system safety.
- 💻 Code Improvement - Suggestions to enhance code quality.
- 🔨 Refactor Suggestion - Recommendations for restructuring code.
- ℹ️ Others - General comments and information.
Interact with the Bot:
- Send a message or request using the format:
@entelligenceai + *your message*
Example: @entelligenceai Can you suggest improvements for this code?
- Help the Bot learn by providing feedback on its responses.
@entelligenceai + *feedback*
Example: @entelligenceai Do not comment on `save_auth` function !
Also you can trigger various commands with the bot by doing
@entelligenceai command
The current supported commands are
config- shows the current configretrigger_review- retriggers the review
More commands to be added soon.
| timm==1.0.7 | ||
| torchmetrics==1.4.0 | ||
| pycocotools==2.0.7 | ||
| sentence-transformers==3.0.1 No newline at end of file | ||
| sentence-transformers==3.0.1 | ||
| torch>=2.10.0 # not directly required, pinned by Snyk to avoid a vulnerability | ||
| wheel>=0.46.2 # not directly required, pinned by Snyk to avoid a vulnerability No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correctness: Installation will fail: torch>=2.10.0 does not exist (latest is ~2.3.x). Replace with a real, compatible version like torch>=2.0.1,<2.4.0 after verifying the actual Snyk CVE.
🤖 AI Agent Prompt for Cursor/Windsurf
📋 Copy this prompt to your AI coding assistant (Cursor, Windsurf, etc.) to get help fixing this issue
File: requirements.txt, Lines 54-55
Problem: The dependency `torch>=2.10.0` specifies a non-existent PyTorch version, causing installation failures and conflicts with other ML libraries.
Fix Instructions:
1. Identify the actual Snyk vulnerability CVE for torch
2. Determine the minimum patched torch version that addresses the vulnerability
3. Replace `torch>=2.10.0` with a realistic version constraint (e.g., `torch>=2.0.1,<2.4.0`)
4. Verify compatibility with sentence-transformers==3.0.1, torchmetrics==1.4.0, and other torch-dependent packages
5. Test pip install with the updated requirements.txt to ensure no conflicts
✨ Committable Code Suggestion
💡 This is a one-click fix! Click "Commit suggestion" to apply this change directly to your branch.
| timm==1.0.7 | |
| torchmetrics==1.4.0 | |
| pycocotools==2.0.7 | |
| sentence-transformers==3.0.1 | |
| \ No newline at end of file | |
| sentence-transformers==3.0.1 | |
| torch>=2.10.0 # not directly required, pinned by Snyk to avoid a vulnerability | |
| wheel>=0.46.2 # not directly required, pinned by Snyk to avoid a vulnerability | |
| pyyaml==6.0.1 | |
| timm==1.0.7 | |
| torchmetrics==1.4.0 | |
| pycocotools==2.0.7 | |
| sentence-transformers==3.0.1 | |
| torch>=2.0.1,<2.4.0 # not directly required, pinned by Snyk to avoid a vulnerability | |
| wheel>=0.46.2 # not directly required, pinned by Snyk to avoid a vulnerability |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
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:
🦉 Buffer Overflow
🦉 Directory Traversal
EntelligenceAI PR Summary
Security update pinning indirect dependencies to address vulnerabilities identified by Snyk.
torchto version 2.10.0 or higherwheelto version 0.46.2 or higher