Skip to content

ci: Address new CVEs from rc4 - #1497

Merged
thomasdhc merged 5 commits into
mainfrom
donghyukc/address_new_rc4_cve
Feb 12, 2026
Merged

ci: Address new CVEs from rc4#1497
thomasdhc merged 5 commits into
mainfrom
donghyukc/address_new_rc4_cve

Conversation

@thomasdhc

Copy link
Copy Markdown
Contributor

Description

Usage

# Add snippet demonstrating usage

Checklist

  • I am familiar with the Contributing Guide.
  • New or Existing tests cover these changes.
  • The documentation is up to date with these changes.

Signed-off-by: Dong Hyuk Chang <donghyukc@nvidia.com>
@thomasdhc thomasdhc added the r1.1.0 Pick this label for auto cherry-picking into r1.1.0 label Feb 12, 2026
@greptile-apps

greptile-apps Bot commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

Greptile Overview

Greptile Summary

This PR addresses multiple security vulnerabilities by upgrading dependencies to patched versions.

Key Changes:

  • Dockerfile: Added gnupg upgrade to address CVE-2025-68973 with proper cleanup commands (apt-get clean and removal of /var/lib/apt/lists/*)
  • Python Dependencies: Updated minimum version constraints for four critical packages:
  • Lock File: Automatically regenerated to reflect new constraints with updated package hashes and wheel URLs

All changes are security-focused dependency version bumps following standard CVE remediation practices.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • All changes are straightforward security dependency upgrades following established CVE remediation patterns. The Dockerfile change properly upgrades gnupg with appropriate cleanup. The Python dependency constraints are minimal version bumps (patch/minor versions) that address specific CVEs. The lock file changes are automatic and correctly reflect the constraint updates with proper hashes.
  • No files require special attention

Important Files Changed

Filename Overview
docker/Dockerfile Added gnupg upgrade to address CVE-2025-68973 with proper cleanup
pyproject.toml Updated version constraints for cryptography, nbconvert, pillow, and protobuf to address multiple CVEs
uv.lock Lock file automatically updated to reflect new dependency version constraints

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment thread docker/Dockerfile Outdated
curl \
git \
vim && \
apt install -y --only-upgrade gnupg && \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider using apt-get install instead of apt install for consistency with the rest of the RUN command and better scripting compatibility.

Suggested change
apt install -y --only-upgrade gnupg && \
apt-get install -y --only-upgrade gnupg && \

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Signed-off-by: Dong Hyuk Chang <donghyukc@nvidia.com>

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@thomasdhc

Copy link
Copy Markdown
Contributor Author

/ok to test 62b1c91

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@ayushdg

ayushdg commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

/ok to test d02fe8f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

r1.1.0 Pick this label for auto cherry-picking into r1.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants