Revert "handle disconnect/hibernation reconnection"

This reverts commit 15300c6.

Author: AnthonyGress

Dockerfile

@@ -20,7 +20,7 @@ FROM node:lts-slim AS backend-deploy
 WORKDIR /app
 ENV NODE_ENV=production
 EXPOSE 2022
-RUN apt-get update && apt-get install -y lm-sensors ca-certificates
+RUN apt-get update && apt-get install -y iputils-ping lm-sensors
 COPY --from=backend-build /usr/src/app/dist/config ../config
 COPY --from=backend-build /usr/src/app/dist/index.js ./
 COPY --from=backend-build /usr/src/app/dist/package.json ./

backend/src/routes/deluge.route.ts

@@ -222,6 +222,7 @@ delugeRoute.post('/login', async (req: Request, res: Response) => {
 delugeRoute.get('/stats', async (req: Request, res: Response) => {
     try {
         const baseUrl = getBaseUrl(req);
+        console.log(`Deluge stats request for ${baseUrl}`);
 
         // Use IP address as identifier for non-authenticated users
         const sessionId = req.user?.username || req.ip || 'default';

frontend/src/App.tsx

@@ -5,7 +5,6 @@ import { useEffect } from 'react';
 import { Route, BrowserRouter as Router, Routes } from 'react-router-dom';
 
 import { DashApi } from './api/dash-api';
-import { ConnectionStatus } from './components/ConnectionStatus';
 import { SetupForm } from './components/forms/SetupForm';
 import { WithNav } from './components/navbar/WithNav';
 import { ScrollToTop } from './components/ScrollToTop';
@@ -103,7 +102,6 @@ export const App = () => {
                         <Route path='/signup' element={<LoginPage />}/>
                     </Route>
                 </Routes>
-                <ConnectionStatus />
             </Router>
         </>
     );

frontend/src/api/dash-api.ts

@@ -90,56 +90,11 @@ export class DashApi {
         // Set withCredentials globally for all API requests
         axios.defaults.withCredentials = true;
 
-        // Variable to track last connection attempt time
-        let lastConnectionAttempt = Date.now();
-
-        // Track if we're processing a connection retry
-        let isRetrying = false;
-
-        // Listen for online/offline events
-        window.addEventListener('online', () => {
-            console.log('Browser reports online status');
-            // Force a connection check when we're back online
-            this.checkCookies().catch(e => console.error('Cookie check after online event failed', e));
-        });
-
         // Request interceptor
         axios.interceptors.request.use(
             async (config) => {
                 // Ensure credentials are sent for all API requests
                 config.withCredentials = true;
-
-                // Check if we need to update the last connection time
-                // This is used to detect when the device has been sleeping
-                const now = Date.now();
-                const timeSinceLastConnection = now - lastConnectionAttempt;
-                lastConnectionAttempt = now;
-
-                // If there's been an unusually long gap between requests (device sleep)
-                // and this isn't a login/refresh/health check request, do a connection check
-                const isAuthRequest = config.url?.includes('api/auth') || config.url?.includes('api/config') ||
-                                     config.url?.includes('api/health');
-
-                if (timeSinceLastConnection > 60000 && !isRetrying && !isAuthRequest) {
-                    console.log(`Long time since last request (${timeSinceLastConnection}ms), checking connection`);
-                    isRetrying = true;
-
-                    // Check connection before proceeding
-                    try {
-                        await this.checkCookies();
-                        isRetrying = false;
-                    } catch (e) {
-                        console.error('Connection check failed, attempting token refresh');
-                        try {
-                            await this.refreshToken();
-                        } catch (refreshError) {
-                            console.error('Token refresh failed after connection check', refreshError);
-                        } finally {
-                            isRetrying = false;
-                        }
-                    }
-                }
-
                 return config;
             },
             (error) => Promise.reject(error)
@@ -149,47 +104,6 @@ export class DashApi {
         axios.interceptors.response.use(
             response => response,
             async (error) => {
-                // Set a default retry delay to add some buffer before retrying
-                const retryDelay = 1000;
-
-                // Handle device wake network issues
-                // Common errors after device wake: ERR_NETWORK, ETIMEDOUT, ECONNABORTED
-                if (
-                    error.code === 'ERR_NETWORK' ||
-                    error.code === 'ECONNABORTED' ||
-                    error.code === 'ETIMEDOUT' ||
-                    error.message?.includes('Network Error')
-                ) {
-                    const originalRequest = error.config;
-
-                    // Avoid infinite retry loops
-                    if (!originalRequest._retry && !isRetrying) {
-                        originalRequest._retry = true;
-                        isRetrying = true;
-
-                        console.log('Network error detected, attempting reconnection...');
-
-                        // Add a delay to give network time to stabilize
-                        await new Promise(resolve => setTimeout(resolve, retryDelay));
-
-                        try {
-                            // Try to refresh connection
-                            const refreshResult = await this.refreshToken();
-                            isRetrying = false;
-
-                            if (refreshResult.success) {
-                                console.log('Connection restored after network error');
-                                // If refresh was successful, retry the original request
-                                return axios(originalRequest);
-                            }
-                        } catch (refreshError) {
-                            console.error('Failed to restore connection', refreshError);
-                        } finally {
-                            isRetrying = false;
-                        }
-                    }
-                }
-
                 const originalRequest = error.config;
 
                 // Only handle 401 errors for authenticated routes

frontend/src/components/ConnectionStatus.tsx

@@ -1,32 +1,14 @@
 import { Box, CardContent, Typography } from '@mui/material';
 import { useEffect, useState } from 'react';
 
-import { useAppContext } from '../../../../context/useAppContext';
-
 export const DateTimeWidget = () => {
-    const [dateTime, setDateTime] = useState<Date>(new Date());
-    const { refreshCounter } = useAppContext();
-
-    useEffect(() => {
-        // Set initial time
-        setDateTime(new Date());
+    const [dateTime, setDateTime] = useState<any>(new Date());
 
-        // Create interval to update time every minute
-        const intervalId = setInterval(() => {
+    useEffect(()=>{
+        setInterval(()=>{
             setDateTime(new Date());
         }, 60000);
-
-        // Clean up interval on component unmount
-        return () => {
-            clearInterval(intervalId);
-        };
-    }, []);
-
-    // Force an immediate refresh when dashboard is refreshed
-    useEffect(() => {
-        // Update time immediately when dashboard refreshes
-        setDateTime(new Date());
-    }, [refreshCounter]);
+    },[]);
 
     return (
         <CardContent>

frontend/src/components/dashboard/base-items/widgets/DateTimeWidget.tsx

@@ -2,7 +2,6 @@ import { useCallback, useEffect, useState } from 'react';
 
 import { TorrentClientWidget } from './TorrentClientWidget';
 import { DashApi } from '../../../../api/dash-api';
-import { useAppContext } from '../../../../context/useAppContext';
 
 type DelugeWidgetConfig = {
     host?: string;
@@ -16,7 +15,6 @@ type DelugeWidgetConfig = {
 
 export const DelugeWidget = (props: { config?: DelugeWidgetConfig }) => {
     const { config } = props;
-    const { refreshCounter } = useAppContext();
     const [isLoading, setIsLoading] = useState(false);
     const [isAuthenticated, setIsAuthenticated] = useState(false);
     const [authError, setAuthError] = useState('');
@@ -170,21 +168,12 @@ export const DelugeWidget = (props: { config?: DelugeWidgetConfig }) => {
             const interval = setInterval(() => {
                 fetchStats();
                 fetchTorrents();
-            }, 30000);
+            }, 5000); // Fixed interval of 5000ms as specified
 
             return () => clearInterval(interval);
         }
     }, [fetchStats, fetchTorrents, isAuthenticated]);
 
-    // Force refresh when dashboard is refreshed
-    useEffect(() => {
-        if (isAuthenticated && !loginAttemptFailed) {
-            console.log('Dashboard refreshed, re-fetching Deluge data');
-            fetchStats();
-            fetchTorrents();
-        }
-    }, [refreshCounter, fetchStats, fetchTorrents, isAuthenticated, loginAttemptFailed]);
-
     // Torrent actions
     const handleResumeTorrent = useCallback(async (hash: string) => {
         try {

frontend/src/components/dashboard/base-items/widgets/DelugeWidget.tsx

@@ -42,7 +42,7 @@ const initialStats: PiholeStats = {
 
 export const PiholeWidget = (props: { config?: PiholeWidgetConfig }) => {
     const { config } = props;
-    const { editMode, refreshCounter } = useAppContext();
+    const { editMode } = useAppContext();
 
     // Reference to track if this is the first render
     const isFirstRender = useRef(true);
@@ -123,14 +123,6 @@ export const PiholeWidget = (props: { config?: PiholeWidgetConfig }) => {
     // eslint-disable-next-line react-hooks/exhaustive-deps
     }, []);
 
-    // Force refresh when dashboard is refreshed
-    useEffect(() => {
-        if (!isFirstRender.current && isConfigured && !error && !authFailed) {
-            console.log('Dashboard refreshed, re-fetching Pi-hole data');
-            checkPiholeStatus();
-        }
-    }, [refreshCounter]);
-
     // Determine if we're using Pi-hole v6 based on authentication method
     useEffect(() => {
         // Using password-only auth indicates Pi-hole v6

frontend/src/components/dashboard/base-items/widgets/PiholeWidget/PiholeWidget.tsx

@@ -2,7 +2,6 @@ import { useCallback, useEffect, useState } from 'react';
 
 import { TorrentClientWidget } from './TorrentClientWidget';
 import { DashApi } from '../../../../api/dash-api';
-import { useAppContext } from '../../../../context/useAppContext';
 
 type QBittorrentWidgetConfig = {
     host?: string;
@@ -17,7 +16,6 @@ type QBittorrentWidgetConfig = {
 
 export const QBittorrentWidget = (props: { config?: QBittorrentWidgetConfig }) => {
     const { config } = props;
-    const { refreshCounter } = useAppContext();
     const [isLoading, setIsLoading] = useState(false);
     const [isAuthenticated, setIsAuthenticated] = useState(false);
     const [authError, setAuthError] = useState('');
@@ -171,21 +169,12 @@ export const QBittorrentWidget = (props: { config?: QBittorrentWidgetConfig }) =
             const interval = setInterval(() => {
                 fetchStats();
                 fetchTorrents();
-            }, 30000);
+            }, 5000); // Fixed interval of 5000ms as specified
 
             return () => clearInterval(interval);
         }
     }, [fetchStats, fetchTorrents, isAuthenticated]);
 
-    // Force refresh when dashboard is refreshed
-    useEffect(() => {
-        if (isAuthenticated) {
-            console.log('Dashboard refreshed, re-fetching qBittorrent data');
-            fetchStats();
-            fetchTorrents();
-        }
-    }, [refreshCounter, fetchStats, fetchTorrents, isAuthenticated]);
-
     // Torrent actions
     const handleStartTorrent = useCallback(async (hash: string) => {
         try {