fix(container): bump uv to 0.11.22, remove uv/uvx from runtime#198
fix(container): bump uv to 0.11.22, remove uv/uvx from runtime#198SudipSinha wants to merge 1 commit into
Conversation
Resolves GHSA-4gg8-gxpx-9rph (arbitrary file write through entry point names) by upgrading uv from 0.11.1 to 0.11.22 (fix version: 0.11.15). Also explicitly removes uv/uvx binaries from the builder's bin directory before copying to the runtime stage, since they are not needed at runtime and their presence unnecessarily expands the attack surface. Signed-off-by: Sudip Sinha <Sudip.Sinha@RedHat.com>
|
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)
📝 WalkthroughWalkthroughThe uv Version Bump
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 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)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
PR image build and manifest generation completed successfully! 📦 PR image: 🗂️ CI manifests |
Summary
uvfrom 0.11.1 to 0.11.22 in the Containerfile builder stage (fix version: 0.11.15)uv/uvxbinaries from the builder'sbin/directory before copying to the runtime stage — they are not needed at runtime and their presence unnecessarily expands the attack surfaceSecurity
Resolves GHSA-4gg8-gxpx-9rph — arbitrary file write through entry point names in uv < 0.11.15.
Test plan
podman build -t trustyai:test --build-arg EXTRAS="mariadb,protobuf,eval" .uv/uvxare absent from runtime:podman run --rm trustyai:test ls -la /opt/app-root/bin/ | grep -E 'uv$|uvx$'should return nothing🤖 Generated with Claude Code
Summary by CodeRabbit