Skip to content

fix(ui): redirect to login on 401 token_not_found_in_db #38

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

Merged
songkuan-zheng merged 2 commits into from
Jun 3, 2026
Merged

fix(ui): redirect to login on 401 token_not_found_in_db #38

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fff61cb
fix(ui): redirect to login on 401 token_not_found_in_db (and siblings)
songkuan-zheng Jun 3, 2026
ff82a41
fix(ui): handle MCP tool call errorData scope after Patch 1 sed sweep
songkuan-zheng Jun 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/accessGroups/useAccessGroupDetails.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ const fetchAccessGroupDetails = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
5 changes: 4 additions & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/accessGroups/useAccessGroups.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,10 @@ describe("useAccessGroups", () => {
expect(result.current.error).toBeInstanceOf(Error);
expect((result.current.error as Error).message).toBe("Forbidden");
expect(result.current.data).toBeUndefined();
expect(networking.handleError).toHaveBeenCalledWith("Forbidden");
// handleError now receives the raw error object (not the derived string)
// so its built-in `extractErrorType` can dispatch on `error.type` for
// auth-related redirects (D1 taxonomy). See networking.tsx:507.
expect(networking.handleError).toHaveBeenCalledWith({ detail: "Forbidden" });
});

it("should return empty array when API returns empty list", async () => {
Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/accessGroups/useAccessGroups.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ const fetchAccessGroups = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/accessGroups/useCreateAccessGroup.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ const createAccessGroup = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/accessGroups/useDeleteAccessGroup.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ const deleteAccessGroup = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/accessGroups/useEditAccessGroup.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ const updateAccessGroup = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/guardrails/useRegisterGuardrail.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ const registerGuardrail = async (
if (!response.ok) {
const errorData = await response.json().catch(() => ({}));
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/keys/useKeys.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ const keyListCall = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/keys/useResetKeySpend.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export const resetKeySpend = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/projects/useCreateProject.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ const createProject = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/projects/useDeleteProject.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ const deleteProjects = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}
};
Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/projects/useProjectDetails.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ const fetchProjectDetails = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/projects/useProjects.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ const fetchProjects = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
2 changes: 1 addition & 1 deletion ui/litellm-dashboard/src/app/(dashboard)/hooks/projects/useUpdateProject.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ const updateProject = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
4 changes: 2 additions & 2 deletions ui/litellm-dashboard/src/app/(dashboard)/hooks/proxyConfig/useProxyConfig.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ export const getProxyConfigCall = async (accessToken: string, configType: Config
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down Expand Up @@ -128,7 +128,7 @@ export const deleteProxyConfigFieldCall = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
4 changes: 2 additions & 2 deletions ui/litellm-dashboard/src/app/(dashboard)/hooks/teams/useTeams.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ export const teamListCall = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down Expand Up @@ -202,7 +202,7 @@ const deletedTeamListCall = async (
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
handleError(errorData);
throw new Error(errorMessage);
}

Expand Down
Loading