reverted back the unnecessary code

Author: Rajat-Dabade

server/api/api.go

@@ -62,12 +62,6 @@ func NewAPI(
 }
 
 func (a *API) RegisterRoutes(r *mux.Router) {
-	// V1 routes for simple endpoints
-	apiv1 := r.PathPrefix("/api/v1").Subrouter()
-	apiv1.Use(a.panicHandler)
-	apiv1.Use(a.requireCSRFToken)
-	a.registerKeepaliveRoute(apiv1)
-
 	apiv2 := r.PathPrefix("/api/v2").Subrouter()
 	apiv2.Use(a.panicHandler)
 	apiv2.Use(a.requireCSRFToken)
@@ -233,20 +227,3 @@ func setResponseHeader(w http.ResponseWriter, key string, value string) { //noli
 	}
 	header.Set(key, value)
 }
-
-// registerKeepaliveRoute registers the keepalive endpoint for session management.
-func (a *API) registerKeepaliveRoute(r *mux.Router) {
-	r.HandleFunc("/keepalive", a.handleKeepalive).Methods("GET")
-}
-
-// handleKeepalive handles the keepalive request to maintain session.
-// The main purpose is to keep the session alive and prevent WebSocket staleness.
-func (a *API) handleKeepalive(w http.ResponseWriter, r *http.Request) {
-	userID := getUserID(r)
-	if userID == "" {
-		a.errorResponse(w, r, model.NewErrUnauthorized("unauthorized"))
-		return
-	}
-	a.logger.Debug("Keepalive request received", mlog.String("userID", userID))
-	jsonStringResponse(w, http.StatusOK, `{"status":"ok"}`)
-}

webapp/src/index.tsx

@@ -167,7 +167,6 @@ export default class Plugin {
     rhsId?: string
     boardSelectorId?: string
     registry?: PluginRegistry
-    activityFunc?: () => void
 
     // eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-empty-function
     async initialize(registry: PluginRegistry, mmStore: Store<GlobalState, Action<Record<string, unknown>>>): Promise<void> {
@@ -231,8 +230,6 @@ export default class Plugin {
             })
         }
 
-        this.userActivityWatch()
-
         let lastViewedChannel = mmStore.getState().entities.channels.currentChannelId
         let prevTeamID: string
 
@@ -394,36 +391,6 @@ export default class Plugin {
         }
     }
 
-    userActivityWatch(): void {
-        let lastActivityTime = Number.MAX_SAFE_INTEGER
-        const activityTimeout = 60 * 60 * 1000 // 1 hour
-
-        this.activityFunc = () => {
-            const now = new Date().getTime()
-            if (now - lastActivityTime > activityTimeout) {
-                this.notifyConnect()
-            }
-            lastActivityTime = now
-        }
-        document.addEventListener('click', this.activityFunc)
-    }
-
-    async notifyConnect(): Promise<void> {
-        try {
-            const response = await fetch(`${windowAny.baseURL}/api/v1/keepalive`, {
-                method: 'GET',
-                headers: {
-                    'X-Timezone-Offset': (-new Date().getTimezoneOffset() / 60).toString(),
-                },
-            })
-            if (!response.ok) {
-                Utils.logError(`Keep-alive request failed: ${response.status}`)
-            }
-        } catch (error) {
-            Utils.logError(`Keep-alive request error: ${error}`)
-        }
-    }
-
     uninitialize(): void {
         if (this.channelHeaderButtonId) {
             this.registry?.unregisterComponent(this.channelHeaderButtonId)
@@ -435,11 +402,6 @@ export default class Plugin {
             this.registry?.unregisterComponent(this.boardSelectorId)
         }
 
-        // Clean up activity watch
-        if (this.activityFunc) {
-            document.removeEventListener('click', this.activityFunc)
-        }
-
         // unregister websocket handlers
         this.registry?.unregisterWebSocketEventHandler(wsClient.clientPrefix + ACTION_UPDATE_BLOCK)
     }