Skip to content

fix: Exclude litellm==1.107.0 from extras - #8104

Merged
sentrivana merged 1 commit into
getsentry:masterfrom
jonny-norwine:exclude-litellm-11070-from-extras
Oct 22, 2026
Merged

sentrivana merged 1 commit into
getsentry:masterfrom
jonny-norwine:exclude-litellm-11070-from-extras

Conversation

@jonny-norwine

Copy link
Copy Markdown
Contributor

Description

That version is vulnerable so it would be great to exclude it from extras. To be honest, we don't use that extra but since we use poetry, the litellm version range shows up in our poetry.lock and it doesn't look good for our security team. Thanks!

Issues

Reminders

@jonny-norwine
jonny-norwine requested a review from a team as a code owner October 21, 2026 20:14

@chrikrah chrikrah left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@jonny-norwine I'd merge this at 3f28707. It follows #5876, which excluded 1.82.7 and 1.82.8 the same way, and the specifier drops only 1.107.0. The rc builds that the SafeDep write-up calls clean still resolve.

$ python3 spec_check.py   # packaging SpecifierSet(...).contains(v, prereleases=True) on the litellm extra in setup.py at 3f28707
setup.py at 3f28707: litellm>=1.77.5,!=1.82.7,!=1.82.8,!=1.107.0
1.82.7 False
1.106.2 True
1.107.0rc2 True
1.107.0 False
1.107.1 True

$ curl -s https://pypi.org/pypi/litellm/json | python3 releases.py   # the releases map, 2026-10-21
1.82.7 absent
1.82.8 absent
1.107.0 absent
latest 1.106.2

PyPI has already removed 1.107.0, so a fresh resolve from pypi.org cannot pick it. The exclusion still guards mirrors that cached it before the removal, and it cleans up the range a scanner reads from your poetry.lock.

@sentrivana this is one line in setup.py. Could it go out in the next patch release, so that lock files pick up the exclusion?

@sentrivana sentrivana left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you @jonny-norwine!

@sentrivana
sentrivana enabled auto-merge (squash) October 22, 2026 07:19
@sentrivana
sentrivana merged commit 25d265b into getsentry:master Oct 22, 2026
161 checks passed
@jonny-norwine

Copy link
Copy Markdown
Contributor Author

Thank you both

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.

Exclude litellm==1.107.0 from the extras

3 participants