fix(deps): vuln protobuf (major → 7.35.1) [apps/rest-services/py] - #296
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2c313c106e
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| opentelemetry-util-http==0.41b0 | ||
| packaging==23.2 | ||
| protobuf==4.24.4 | ||
| protobuf==7.35.1 |
There was a problem hiding this comment.
Keep protobuf within OpenTelemetry's supported range
In this requirements file, opentelemetry-proto==1.20.0 is still pinned and that package declares protobuf<5.0,>=3.19, so pinning protobuf==7.35.1 makes fresh installs of apps/rest-services/py/requirements.txt unsatisfiable with pip's resolver. Any setup following this sample's README will fail before the app can run; either use a patched <5 protobuf release or upgrade the OpenTelemetry packages together.
Useful? React with 👍 / 👎.
Summary: High-severity security update — 1 package upgraded (MAJOR changes included)
Manifests changed:
apps/rest-services/py(pip)✅ Action Required: Please review the changes below. If they look good, approve and merge this PR.
Updates
Warning
Major Version Upgrade
This update includes major version changes that may contain breaking changes. Please:
Security Details
🚨 Critical & High Severity (6 fixed)
4.24.47.35.1apps/rest-services/py/requirements.txtReview Checklist
Extra review is recommended for this update:
Update Mode: all_vulns
🤖 Generated by DataDog Automated Dependency Management System