Skip to content

[Refactor] /login route - #17379

Merged
yuneng-jiang merged 3 commits into
mainfrom
litellm_login_route_refactor
Dec 3, 2025
Merged

yuneng-jiang merged 3 commits into
mainfrom
litellm_login_route_refactor

Conversation

@yuneng-jiang

@yuneng-jiang yuneng-jiang commented Dec 2, 2025 •

Copy link
Copy Markdown
Contributor

[Refactor] /login route

Relevant issues

This PR refactors the /login route's logic to another file, aiming to reuse most of this logic for a v2/login. More information about v2/login in a subsequent PR. This PR should not change the behavior of the currently login.

Tested by logging in with these credentials:

  1. Default Proxy Admin (admin + master key)
  2. Other Proxy Admin
  3. Team Admin
  4. Internal User
  5. Internal Viewer
  6. Bad Credentials

All verified the behavior is the same as before

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

🧹 Refactoring
✅ Test

Changes

image image

@vercel

vercel Bot commented Dec 2, 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 Dec 3, 2025 7:02pm

@yuneng-jiang
yuneng-jiang merged commit dcb7552 into main Dec 3, 2025
38 of 59 checks passed
@ishaan-berri
ishaan-berri deleted the litellm_login_route_refactor branch March 26, 2026 21:54
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 2026

This branch was successfully deployed

1 active deployment
Preview — e6620fcd Deployed Dec 3, 2025 by vercel[bot]
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