From 5e80234379960e41293b3eb3645fb9bd0a07f000 Mon Sep 17 00:00:00 2001 From: marwanakeel Date: Mon, 16 Feb 2026 14:52:09 +0400 Subject: [PATCH] fix: add safety limit to pagination loop and hoist baseUrl - Add MAX_PAGES=50 guard to prevent unbounded fetching (caps at 5,000 logs) - Move baseUrl computation outside the loop since it's invariant Co-Authored-By: Claude Opus 4.6 --- ui/litellm-dashboard/src/components/networking.tsx | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/ui/litellm-dashboard/src/components/networking.tsx b/ui/litellm-dashboard/src/components/networking.tsx index 390f04af627..77c94c5d342 100644 --- a/ui/litellm-dashboard/src/components/networking.tsx +++ b/ui/litellm-dashboard/src/components/networking.tsx @@ -7026,15 +7026,17 @@ export const teamPermissionsUpdateCall = async (accessToken: string, teamId: str */ export const sessionSpendLogsCall = async (accessToken: string, session_id: string) => { const pageSize = 100; // backend maximum + const maxPages = 50; // safety limit — caps at 5,000 logs let currentPage = 1; let allData: any[] = []; let total = 0; + const baseUrl = proxyBaseUrl + ? `${proxyBaseUrl}/spend/logs/session/ui` + : `/spend/logs/session/ui`; + try { - while (true) { - const baseUrl = proxyBaseUrl - ? `${proxyBaseUrl}/spend/logs/session/ui` - : `/spend/logs/session/ui`; + while (currentPage <= maxPages) { const url = `${baseUrl}?session_id=${encodeURIComponent(session_id)}&page=${currentPage}&page_size=${pageSize}`; const response = await fetch(url, {