[Snyk] Security upgrade protobuf from 4.24.4 to 6.33.5#60
[Snyk] Security upgrade protobuf from 4.24.4 to 6.33.5#60pratham-aggr wants to merge 1 commit intomainfrom
Conversation
The following vulnerabilities are fixed by pinning transitive dependencies: - https://snyk.io/vuln/SNYK-PYTHON-PROTOBUF-15090738
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
This PR is being reviewed by Cursor Bugbot
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.
| matplotlib==3.7.2 | ||
| seaborn==0.12.2 | ||
| websockets>=13.0 | ||
| protobuf>=6.33.5 # not directly required, pinned by Snyk to avoid a vulnerability |
There was a problem hiding this comment.
Major protobuf version upgrade may break yfinance compatibility
Medium Severity
The upgrade from protobuf 4.24.4 to 6.33.5 represents a 2-major-version jump that may introduce compatibility issues with yfinance==0.2.65, which depends on protobuf. The Snyk warning in the PR description indicates potential dependency concerns. This codebase heavily uses yfinance for market data fetching. If yfinance has an upper-bound version constraint on protobuf (e.g., <5.0), this change could cause dependency resolution failures or runtime errors. This compatibility should be verified before merging.


Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):
backend-api/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
Low Risk
Single dependency constraint change; low risk aside from potential upstream incompatibility with packages that expect older
protobufversions.Overview
Pins
protobufto>=6.33.5inbackend-api/requirements.txtto address a Snyk-reported vulnerability (even though it’s an indirect dependency, e.g., viayfinance).Written by Cursor Bugbot for commit 5a13c98. This will update automatically on new commits. Configure here.