Skip to content

[Feature] UI - Org Admin Team Permissions Fix - #17110

Merged
yuneng-jiang merged 1 commit into
mainfrom
litellm_org_admin_access_fix
Nov 26, 2025
Merged

[Feature] UI - Org Admin Team Permissions Fix#17110
yuneng-jiang merged 1 commit into
mainfrom
litellm_org_admin_access_fix

Conversation

@yuneng-jiang

Copy link
Copy Markdown
Contributor

[Feature] UI - Org Admin Team Permissions Fix

Relevant issues

When the model select is empty, this is perceived as "All proxy models" by the server. This opens up the possibility of the following:

  1. An org admin has access to a subset of the proxy models
  2. The org admin creates a team with access to their subset of models
  3. The org admin modifies the team models to the empty state, giving the team access to all proxy models

The situation above allows the org admin to give a team all proxy model access, even though none of the parties involved should have access to all proxy models.

This PR fixes this issue by making the model select required (needs at least 1 entry to be submit-able). The "All Proxy Models" option only appears if the user is a proxy admin or has access to "all-proxy-models"

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • I have added a screenshot of my new test passing locally
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

Type

🆕 New Feature
✅ Test

Changes

image image image

@vercel

vercel Bot commented Nov 26, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
litellm Ready Ready Preview Comment Nov 26, 2025 1:03am

@yuneng-jiang
yuneng-jiang merged commit 38bac31 into main Nov 26, 2025
33 of 59 checks passed
@ishaan-berri
ishaan-berri deleted the litellm_org_admin_access_fix branch March 26, 2026 21:54
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 2026
…s_fix

[Feature] UI - Org Admin Team Permissions Fix
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.

1 participant