Added page-not-found element and improved secured component with replace, and simplified

pac-man-board-game/ClientApp/src/App.tsx

@@ -1,5 +1,5 @@
-import React, {FC, useEffect} from "react";
+import React, {FC} from "react";
-import {Route, Routes, useNavigate} from "react-router-dom";
+import {Navigate, Route, Routes} from "react-router-dom";
 import Layout from "./components/layout";
 import AppRoutes from "./AppRoutes";
 import "./index.css";
@@ -23,20 +23,14 @@ export const App: FC = () => (
  * @param secured Whether or not the page is secured.
  * @constructor The Secured component.
  */
-const Secured: FC<{ secured: boolean } & ChildProps> = ({children, secured}) => {
+const Secured: FC<{
+  secured: boolean
+} & ChildProps> = ({children, secured}) => {
   const player = useAtomValue(thisPlayerAtom);
-  const navigate = useNavigate();
-  const redirect = secured && player === undefined
 
-  useEffect(() => {
+  if (secured && player === undefined) {
-    if (redirect) {
+    return <Navigate to={"/login"} replace/>
-      navigate("/login");
-    }
-  });
-
-  if (!redirect) {
-    return (
-      <>{children}</>
-    )
   }
+
+  return <>{children}</>
 }

pac-man-board-game/ClientApp/src/AppRoutes.tsx

@@ -27,6 +27,10 @@ const AppRoutes = [
   {
     path: "/login",
     element: <LoginPage/>
+  },
+  {
+    path: "*",
+    element: <p>Page not found</p>
   }
 ];
 

pac-man-board-game/Controllers/GenericController.cs

@@ -52,7 +52,8 @@ public abstract class GenericController : ControllerBase
             } while (true);
 
             var disconnectSegment = Disconnect();
-            if (disconnectSegment != null) SendDisconnectMessage((ArraySegment<byte>)disconnectSegment);
+            if (disconnectSegment != null) 
+                SendDisconnectMessage((ArraySegment<byte>)disconnectSegment);
 
             await _webSocketService.Close(WebSocket, result.CloseStatus.Value, result.CloseStatusDescription);
         }