CORE-1048: pin minimum versions for msgpack and cryptography to fix Dependabot alerts#2274
Conversation
…ependabot alerts - msgpack >=1.2.1 (fixes OOB read/crash on Unpacker reuse after caught error) - cryptography >=48.0.1 (fixes vulnerable OpenSSL included in wheels) Both are transitive dependencies pinned following the same pattern as idna. Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
👋 @haritamar |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughTwo transitive dependencies, ChangesSecurity Dependency Pins
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
Add minimum version constraints for two transitive dependencies with known Dependabot high-severity alerts:
>=1.2.1>=48.0.1Follows the same pattern used for
idna = \">=3.15,<4\"(CVE-2025-46816). No lockfile in this repo, so pinning inpyproject.tomlis the only way to enforce the minimum safe version.Link to Devin session: https://app.devin.ai/sessions/7a118cfb43a8447e839fbaeace279467
Requested by: @haritamar
Summary by CodeRabbit