Skip to content

Pin lightning <=2.6.1 to avoid CVE in 2.6.2/2.6.3#12

Merged
smcolby merged 2 commits into
mainfrom
fix/pin-lightning-2.6.1-security
May 3, 2026
Merged

Pin lightning <=2.6.1 to avoid CVE in 2.6.2/2.6.3#12
smcolby merged 2 commits into
mainfrom
fix/pin-lightning-2.6.1-security

Conversation

@smcolby
Copy link
Copy Markdown
Collaborator

@smcolby smcolby commented May 1, 2026

PyTorch Lightning 2.6.2 and 2.6.3 contain a security vulnerability. Cap the upper bound so fresh installs don't resolve to affected releases while preserving the existing >=2.0 floor.

Change: "lightning>=2.0" → "lightning>=2.0,<=2.6.1" in pyproject.toml

Note: Revisit once a patched release (≥2.6.4 or new minor/major) ships — the upper bound can then be relaxed or converted to an exclusion range (!=2.6.2,!=2.6.3).

PyTorch Lightning 2.6.2 and 2.6.3 contain a security vulnerability.
Cap the upper bound so fresh installs don't resolve to affected releases
while preserving the existing >=2.0 floor.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@smcolby smcolby self-assigned this May 1, 2026
@smcolby smcolby requested a review from Copilot May 1, 2026 21:57
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Caps the lightning dependency to avoid resolving to vulnerable PyTorch Lightning releases while keeping the existing lower bound, preventing fresh installs from pulling affected versions.

Changes:

  • Update lightning dependency constraint from >=2.0 to >=2.0,<=2.6.1.

Comment thread pyproject.toml Outdated
Copy link
Copy Markdown

@khuddzu khuddzu left a comment

Choose a reason for hiding this comment

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

LGTM

@smcolby smcolby merged commit 822677d into main May 3, 2026
10 checks passed
@smcolby smcolby deleted the fix/pin-lightning-2.6.1-security branch May 3, 2026 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants