Refactor WebAuthnController and related services for improved logging and error handling

- Reorganized imports in WebAuthnController for clarity.
- Enhanced logging functionality in createLog method.
- Updated registration and authentication options handling.
- Improved error handling in registration and authentication processes.
- Refactored webauthnService for better structure and readability.
- Cleaned up middleware imports for consistency.
- Adjusted repository imports to maintain a clean structure.
- Standardized service imports across various service files.
- Removed unnecessary comments and improved code formatting.
- Ensured consistent use of optional chaining and nullish coalescing where applicable.

Author: fox3000foxy

__tests__/index.test.js

@@ -1,70 +1,13 @@
-/* eslint-disable @typescript-eslint/no-require-imports */
-/* eslint-disable no-undef */
-
-/**
- * @file Jest test suite for the application.
- * @module __tests__/index.test
- */
-
-/**
- * Jest test suite for the application.
- *
- * @description
- * This test suite verifies the application's basic setup, including server definition,
- * port configuration, container initialization, and service bindings.
- *
- * Jest Methods Memo:
- * - `describe(name, fn)`: Creates a block that groups together several related tests.
- * - `it(name, fn)`: Defines a single test case with a descriptive name. Alias: `test(name, fn)`.
- * - `expect(value)`: Creates an assertion about a value.
- * - `toBe(value)`: Matches primitive values or checks referential identity.
- * - `toEqual(value)`: Matches when two objects have the same value.  For deep equality, especially useful for objects and arrays.
- * - `toBeDefined()`: Asserts that a value is not `undefined`.
- * - `toBeUndefined()`: Asserts that a value is `undefined`.
- * - `toBeNull()`: Asserts that a value is `null`.
- * - `toBeTruthy()`: Asserts that a value is truthy.
- * - `toBeFalsy()`: Asserts that a value is falsy.
- * - `toBeGreaterThan(number)`: Asserts that a value is greater than a number.
- * - `toBeLessThan(number)`: Asserts that a value is less than a number.
- * - `toBeGreaterThanOrEqual(number)`: Asserts that a value is greater than or equal to a number.
- * - `toBeLessThanOrEqual(number)`: Asserts that a value is less than or equal to a number.
- * - `toContain(item)`: Asserts that an array or string contains a particular item.
- * - `toMatch(regexp)`: Asserts that a string matches a regular expression.
- * - `toThrow(error)`: Asserts that a function throws an error.
- * - `beforeEach(fn, timeout)`: Function to run before each test in the file.
- * - `afterEach(fn, timeout)`: Function to run after each test in the file.
- * - `beforeAll(fn, timeout)`: Function to run once before all tests in the file.
- * - `afterAll(fn, timeout)`: Function to run once after all tests in the file.
- * - `jest.fn()`: Creates a mock function.
- * - `jest.spyOn(object, methodName)`: Spies on a method of an object.
- * - `jest.mock(moduleName, factory, options)`: Mocks a module.
- */
-
-const app = require("../dist/app").app;
-// const container = require("../dist/container").default;
-// const request = require("supertest");
-
+
+
+const app = require("../dist/app").app;
+
 test("Simple assertion", () => {
   expect(true).toBe(true);
-});
-
+});
+
 test("Server definition", () => {
   expect(app).toBeDefined();
-});
-
-// test("Exemple request to /exemple", async () => {
-//     const response = await request(app).get("/exemple");
-//     const expected = container.get("ExempleService").getHelloWorld();
-//     console.log("response", response);
-//     expect(response.statusCode).toBe(200);
-//     expect(response.text).toEqual(expected);
-// });
-
+});
+
 
-// test("Exemple request to /exemple/user", async () => {
-//     const response = await request(app).get("/exemple/user");
-//     const expected = container.get("ExempleService").getHelloWorldWithName("user");
-//     console.log("response", response);
-//     expect(response.statusCode).toBe(200);
-//     expect(response.text).toEqual(expected);
-// });

dist/app.d.ts

@@ -1,19 +1,19 @@
-import "reflect-metadata";
 import express from "express";
+import "reflect-metadata";
+import "./controllers/AuthenticatorController";
+import "./controllers/BuyOrderController";
 import "./controllers/DescribeController";
 import "./controllers/GameController";
+import "./controllers/GameGiftController";
 import "./controllers/InventoryController";
 import "./controllers/ItemController";
 import "./controllers/LobbyController";
-import "./controllers/TradeController";
-import "./controllers/UserController";
+import "./controllers/MarketListingController";
 import "./controllers/OAuth2Controller";
-import "./controllers/StudioController";
 import "./controllers/SearchController";
 import "./controllers/StripeController";
+import "./controllers/StudioController";
+import "./controllers/TradeController";
+import "./controllers/UserController";
 import "./controllers/WebAuthnController";
-import "./controllers/AuthenticatorController";
-import "./controllers/GameGiftController";
-import "./controllers/MarketListingController";
-import "./controllers/BuyOrderController";
 export declare const app: express.Application;

dist/app.js

@@ -59,10 +59,11 @@ server.setConfig((app) => {
     app.use((0, cors_1.default)());
     app.use(express_1.default.static(path.join(__dirname, "public")));
 });
-// 404 handler
+
 server.setErrorConfig((app) => {
     app.use((req, res) => {
         res.status(404).json({ message: "Not Found" });
     });
 });
 exports.app = server.build();
+

dist/container.js

@@ -33,3 +33,4 @@ container.bind("GameGiftService").to(GameGiftService_1.GameGiftService).inSingle
 container.bind("MarketListingService").to(MarketListingService_1.MarketListingService).inSingletonScope();
 container.bind("BuyOrderService").to(BuyOrderService_1.BuyOrderService).inSingletonScope();
 exports.default = container;
+

dist/controllers/AuthenticatorController.d.ts

@@ -1,7 +1,7 @@
 import { Request, Response } from "express";
 import { AuthenticatedRequest } from "../middlewares/LoggedCheck";
-import { IUserService } from "../services/UserService";
 import { ILogService } from "../services/LogService";
+import { IUserService } from "../services/UserService";
 export declare class Authenticator {
     private userService;
     private logService;

dist/controllers/AuthenticatorController.js

@@ -43,7 +43,7 @@ const qrcode = __importStar(require("qrcode"));
 const time2fa_1 = require("time2fa");
 const GenKey_1 = require("../utils/GenKey");
 const Jwt_1 = require("../utils/Jwt");
-// --- UTILS ---
+
 function handleError(res, error, message, status = 500) {
     const msg = error instanceof Error ? error.message : String(error);
     res.status(status).send({ message, error: msg });
@@ -186,3 +186,4 @@ exports.Authenticator = Authenticator = __decorate([
     __param(1, (0, inversify_1.inject)("LogService")),
     __metadata("design:paramtypes", [Object, Object])
 ], Authenticator);
+

dist/controllers/BuyOrderController.d.ts

@@ -1,7 +1,7 @@
 import { Response } from "express";
+import { AuthenticatedRequest } from "../middlewares/LoggedCheck";
 import { IBuyOrderService } from "../services/BuyOrderService";
 import { IItemService } from "../services/ItemService";
-import { AuthenticatedRequest } from "../middlewares/LoggedCheck";
 import { ILogService } from "../services/LogService";
 export declare class BuyOrderController {
     private buyOrderService;
@@ -14,3 +14,4 @@ export declare class BuyOrderController {
     getBuyOrdersByUser(req: AuthenticatedRequest, res: Response): Promise<Response<any, Record<string, any>> | undefined>;
     getActiveBuyOrdersForItem(req: AuthenticatedRequest, res: Response): Promise<void>;
 }
+

dist/controllers/BuyOrderController.js

@@ -16,7 +16,7 @@ exports.BuyOrderController = void 0;
 const inversify_1 = require("inversify");
 const inversify_express_utils_1 = require("inversify-express-utils");
 const LoggedCheck_1 = require("../middlewares/LoggedCheck");
-// --- UTILS ---
+
 function handleError(res, error, message, status = 500) {
     const msg = error instanceof Error ? error.message : String(error);
     res.status(status).send({ message, error: msg });
@@ -143,3 +143,4 @@ exports.BuyOrderController = BuyOrderController = __decorate([
     __param(2, (0, inversify_1.inject)("LogService")),
     __metadata("design:paramtypes", [Object, Object, Object])
 ], BuyOrderController);
+

dist/controllers/DescribeController.js

@@ -27,3 +27,4 @@ __decorate([
 exports.DescribeController = DescribeController = __decorate([
     (0, inversify_express_utils_1.controller)("/describe")
 ], DescribeController);
+

dist/controllers/GameController.d.ts

@@ -1,7 +1,7 @@
 import { Request, Response } from "express";
+import { AuthenticatedRequest } from "../middlewares/LoggedCheck";
 import { IGameService } from "../services/GameService";
 import { ILogService } from "../services/LogService";
-import { AuthenticatedRequest } from "../middlewares/LoggedCheck";
 import { IUserService } from "../services/UserService";
 export declare class Games {
     private gameService;