GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix: Exclude litellm==1.107.0 from extras by jonny-norwine · Pull Request #8104 · getsentry/sentry-python

Conversation

@jonny-norwine

@jonny-norwine jonny-norwine commented Oct 21, 2026

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

  • resolves: Exclude litellm==1.107.0 from the extras #8103
  • resolves: PY-3035

Reminders

  • Please add tests to validate your changes, and lint your code using uv run ruff.
  • Add GH Issue ID & Linear ID (if applicable)
  • PR title should use conventional commit style (feat:, fix:, ref:, meta:)
  • For external contributors: CONTRIBUTING.md, Sentry SDK development docs, Discord community

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@jonny-norwine
Exclude litellm==1.107.0 from extras
3f28707
@jonny-norwine
jonny-norwine requested a review from a team as a code owner October 21, 2026 20:14
chrikrah
chrikrah reviewed Oct 22, 2026
View reviewed changes

@chrikrah chrikrah left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

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?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
sentrivana
sentrivana approved these changes Oct 22, 2026
View reviewed changes

@sentrivana sentrivana left a comment

Copy link
Copy Markdown
Collaborator

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Thank you @jonny-norwine!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@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

Uh oh!

There was an error while loading. Please reload this page.

@sentry-release-bot sentry-release-bot Bot mentioned this pull request Oct 22, 2026
publish: getsentry/[email protected] getsentry/publish#9970
Closed
6 tasks done
@jonny-norwine

jonny-norwine commented Oct 22, 2026

Copy link
Copy Markdown
Contributor Author

Thank you both

chrikrah reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@sentry-release-bot sentry-release-bot Bot mentioned this pull request Oct 27, 2026
publish: getsentry/[email protected] getsentry/publish#10029
Closed
6 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sentrivana sentrivana sentrivana approved these changes
+1 more reviewer
@chrikrah chrikrah chrikrah left review comments
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Exclude litellm==1.107.0 from the extras

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@jonny-norwine @chrikrah @sentrivana
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.