Skip to content

[Fix] /user/new Privilege Escalation#19116

Merged
yuneng-jiang merged 1 commit intomainfrom
litellm_org_admin_escalte
Jan 15, 2026
Merged

[Fix] /user/new Privilege Escalation#19116
yuneng-jiang merged 1 commit intomainfrom
litellm_org_admin_escalte

Conversation

@yuneng-jiang
Copy link
Collaborator

Relevant issues

Currently, when an org admin (internal user) calls /user/new to create a new proxy_admin, the call is successful and a new proxy_admin is created. This is a security vulnerability as internal_users should not be able to create proxy_admins.

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
  • 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

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Type

🐛 Bug Fix
✅ Test

Changes

image image

@vercel
Copy link

vercel bot commented Jan 14, 2026

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

Project Deployment Review Updated (UTC)
litellm Ready Ready Preview, Comment Jan 14, 2026 11:15pm

@yuneng-jiang yuneng-jiang merged commit e071062 into main Jan 15, 2026
46 of 60 checks passed
yuneng-jiang added a commit that referenced this pull request Jan 18, 2026
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