Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Bug]: ui not rendering keys upon relogin #7485

Closed
qte123 opened this issue Dec 31, 2024 · 2 comments · Fixed by #7540
Closed

[Bug]: ui not rendering keys upon relogin #7485

qte123 opened this issue Dec 31, 2024 · 2 comments · Fixed by #7540
Assignees
@krrishdholakia
Labels
bug Something isn't working

Comments

@qte123
Copy link

qte123 commented Dec 31, 2024

What happened?

A bug happened!
The default team interface for virtual keys has an issue where, after successfully creating a key in the default team, logging out causes the key to disappear from the default team. However, attempting to create a key with the same name again results in an error stating that a key with the same name already exists.

If a new team is created in advance, and the key is created within this new team instead, the issue does not occur, and the key remains available upon re-login.

Create Key：
image

re-login:
image
image

Relevant log output

No response

Are you a ML Ops Team?

No

What LiteLLM version are you on ?

v1.56.4

Twitter / LinkedIn details

No response
@qte123 qte123 added the bug Something isn't working label Dec 31, 2024
@bioshazard
Copy link

I just ran into this myself. Exact same behavior. Looks like it is present on litellm:main-v1.56.5 too.

@krrishdholakia
Copy link
Contributor

this looks like a ui issue

based on the error, it appears the key exists in the db

@krrishdholakia krrishdholakia changed the title [Bug]: The issue lies in the default team’s keys not being persisted in the database. [Bug]: ui not rendering keys upon relogin Jan 3, 2025
@krrishdholakia krrishdholakia self-assigned this Jan 3, 2025
krrishdholakia added a commit that referenced this issue Jan 4, 2025
@krrishdholakia
fix(internal_user_endpoints.py): fix sql query to return all keys, in…
c5af281 
…cluding null team id keys on `/user/info`

Fixes #7485
@krrishdholakia krrishdholakia mentioned this issue Jan 4, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@krrishdholakia krrishdholakia

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(router.py): support request prioritization for text completion c… BerriAI/litellm
3 participants
@bioshazard @krrishdholakia @qte123