From 0d1d1f27221d883fbc675edc0f78ccbbf26115e0 Mon Sep 17 00:00:00 2001
From: Jonathan Caicedo <jonathan@jcaicedo.com>
Date: Sat, 23 May 2026 14:20:13 -0400
Subject: [PATCH] fix: revert nginx timeouts; fire stats fetch on mount
 regardless of interval

---
 docker/nginx.conf                           |  2 --
 frontend/src/pages/Stats.jsx                | 16 +++++++---------
 frontend/src/pages/__tests__/Stats.test.jsx | 10 +++++++---
 3 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/docker/nginx.conf b/docker/nginx.conf
index bacfb655..c5cb429e 100644
--- a/docker/nginx.conf
+++ b/docker/nginx.conf
@@ -21,8 +21,6 @@ server {
     location / {
         include uwsgi_params;
         uwsgi_pass unix:/app/uwsgi.sock;
-        uwsgi_read_timeout 300s;
-        uwsgi_send_timeout 300s;
     }
 
     location /assets/ {
diff --git a/frontend/src/pages/Stats.jsx b/frontend/src/pages/Stats.jsx
index ea4ef5f2..86c7f1c3 100644
--- a/frontend/src/pages/Stats.jsx
+++ b/frontend/src/pages/Stats.jsx
@@ -212,19 +212,19 @@ const StatsPage = () => {
     }
   }, [setVodStats]);
 
+  // Always fetch once on mount, regardless of polling interval setting
+  useEffect(() => {
+    fetchChannelStats();
+    fetchVODStats();
+  }, [fetchChannelStats, fetchVODStats]);
+
   // Set up polling for stats when on stats page
   useEffect(() => {
-    const location = window.location;
-    const isOnStatsPage = location.pathname === '/stats';
+    const isOnStatsPage = window.location.pathname === '/stats';
 
     if (isOnStatsPage && refreshInterval > 0) {
       setIsPollingActive(true);
 
-      // Initial fetch
-      fetchChannelStats();
-      fetchVODStats();
-
-      // Set up interval
       const interval = setInterval(() => {
         fetchChannelStats();
         fetchVODStats();
@@ -239,8 +239,6 @@ const StatsPage = () => {
     }
   }, [refreshInterval, fetchChannelStats, fetchVODStats]);
 
-  // Initial fetch is handled by the polling useEffect above (it fetches immediately on mount)
-
   useEffect(() => {
     console.log('Processing channel stats:', channelStats);
     if (
diff --git a/frontend/src/pages/__tests__/Stats.test.jsx b/frontend/src/pages/__tests__/Stats.test.jsx
index 21cfebf7..92cf8560 100644
--- a/frontend/src/pages/__tests__/Stats.test.jsx
+++ b/frontend/src/pages/__tests__/Stats.test.jsx
@@ -297,19 +297,23 @@ describe('StatsPage', () => {
       vi.useRealTimers();
     });
 
-    it('does not poll when interval is 0', async () => {
+    it('does not poll when interval is 0 but still fetches once on mount', async () => {
       vi.useFakeTimers();
 
       useLocalStorage.mockReturnValue([0, mockSetRefreshInterval]);
       render(<StatsPage />);
 
-      expect(fetchActiveChannelStats).toHaveBeenCalledTimes(0);
+      // Should still fetch once on mount even with interval = 0
+      expect(fetchActiveChannelStats).toHaveBeenCalledTimes(1);
+      expect(getVODStats).toHaveBeenCalledTimes(1);
 
       await act(async () => {
         vi.advanceTimersByTime(10000);
       });
 
-      expect(fetchActiveChannelStats).toHaveBeenCalledTimes(0);
+      // Should not have polled — count stays at 1
+      expect(fetchActiveChannelStats).toHaveBeenCalledTimes(1);
+      expect(getVODStats).toHaveBeenCalledTimes(1);
 
       vi.useRealTimers();
     });