Update Node Version to 20.12.2

.circleci/config.yml

@@ -3,7 +3,7 @@ version: 2.1
 jobs:
   build:
     docker:
-      - image: circleci/node:14.17.6
+      - image: cimg/node:20.12.2
     steps:
       - checkout
       - run:
@@ -12,7 +12,7 @@ jobs:
           command: yarn build
   lint:
     docker:
-      - image: circleci/node:14.17.6
+      - image: cimg/node:20.12.2
     steps:
       - checkout
       - restore_cache:
@@ -27,7 +27,7 @@ jobs:
           command: yarn lint
   test:
     docker:
-      - image: circleci/node:14.17.6
+      - image: cimg/node:20.12.2
         environment:
           RDS_HOST: localhost
           RDS_DATABASE: membership_portal

DataSource.ts

@@ -0,0 +1,20 @@
+import Container from 'typedi';
+import { DataSource } from 'typeorm';
+import { models } from './models';
+
+require('dotenv').config();
+
+export const dataSource = new DataSource({
+  type: 'postgres',
+  host: process.env.RDS_HOST,
+  port: Number(process.env.RDS_PORT),
+  username: process.env.RDS_USER,
+  password: process.env.RDS_PASSWORD,
+  database: process.env.RDS_DATABASE,
+  entities: models,
+  migrations: ['migrations/*.ts'],
+  // synchronize: true, // DO NOT USE IN PRODUCTION, make migrations
+});
+
+// important for dependency injection for repositories
+Container.set(DataSource, dataSource);

Dockerfile

@@ -1,12 +1,12 @@
-FROM node:14.15.3 AS build
+FROM node:20.12.2 AS build
 
 WORKDIR /app
 COPY package.json yarn.lock ./
 RUN yarn install --frozen-lockfile
 COPY . .
 RUN yarn build
 
-FROM node:14.15.3-alpine
+FROM node:20.12.2-alpine
 WORKDIR /app
 COPY --from=build /app /app
 EXPOSE 3000

api/controllers/AdminController.ts

@@ -1,4 +1,5 @@
 import { JsonController, Post, Patch, UploadedFile, UseBefore, ForbiddenError, Body, Get } from 'routing-controllers';
+import { Service } from 'typedi';
 import { UserAuthentication } from '../middleware/UserAuthentication';
 import {
   CreateBonusRequest,
@@ -25,6 +26,7 @@ import { UserModel } from '../../models/UserModel';
 import AttendanceService from '../../services/AttendanceService';
 
 @UseBefore(UserAuthentication)
+@Service()
 @JsonController('/admin')
 export class AdminController {
   private storageService: StorageService;

api/controllers/AttendanceController.ts

@@ -1,4 +1,5 @@
 import { JsonController, Get, Post, UseBefore, Params, ForbiddenError, Body } from 'routing-controllers';
+import { Service } from 'typedi';
 import EmailService from '../../services/EmailService';
 import { UserAuthentication } from '../middleware/UserAuthentication';
 import { AuthenticatedUser } from '../decorators/AuthenticatedUser';
@@ -9,6 +10,7 @@ import PermissionsService from '../../services/PermissionsService';
 import { GetAttendancesForEventResponse, GetAttendancesForUserResponse, AttendEventResponse } from '../../types';
 import { UuidParam } from '../validators/GenericRequests';
 
+@Service()
 @JsonController('/attendance')
 export class AttendanceController {
   private attendanceService: AttendanceService;

api/controllers/AuthController.ts

@@ -1,4 +1,5 @@
 import { JsonController, Params, Body, Get, Post, UseBefore } from 'routing-controllers';
+import { Service } from 'typedi';
 import {
   RegistrationResponse,
   LoginResponse,
@@ -26,6 +27,7 @@ import { AuthenticatedUser } from '../decorators/AuthenticatedUser';
 import { UserModel } from '../../models/UserModel';
 import { EmailParam, AccessCodeParam } from '../validators/GenericRequests';
 
+@Service()
 @JsonController('/auth')
 export class AuthController {
   private userAccountService: UserAccountService;

api/controllers/EventController.ts

@@ -1,6 +1,7 @@
 import {
   JsonController, Get, Patch, Delete, Post, UseBefore, Params, ForbiddenError, QueryParams, UploadedFile, Body,
 } from 'routing-controllers';
+import { Service } from 'typedi';
 import EventService from '../../services/EventService';
 import { UserAuthentication, OptionalUserAuthentication } from '../middleware/UserAuthentication';
 import { AuthenticatedUser } from '../decorators/AuthenticatedUser';
@@ -28,6 +29,7 @@ import {
   SubmitEventFeedbackRequest,
 } from '../validators/EventControllerRequests';
 
+@Service()
 @JsonController('/event')
 export class EventController {
   private eventService: EventService;

api/controllers/FeedbackController.ts

@@ -1,5 +1,6 @@
 import { Body, ForbiddenError, Get, JsonController, Params,
   Patch, Post, UseBefore, QueryParams } from 'routing-controllers';
+import { Service } from 'typedi';
 import { AuthenticatedUser } from '../decorators/AuthenticatedUser';
 import { UserModel } from '../../models/UserModel';
 import PermissionsService from '../../services/PermissionsService';
@@ -14,6 +15,7 @@ import {
 } from '../validators/FeedbackControllerRequests';
 
 @UseBefore(UserAuthentication)
+@Service()
 @JsonController('/feedback')
 export class FeedbackController {
   private feedbackService: FeedbackService;

api/controllers/LeaderboardController.ts

@@ -1,10 +1,12 @@
 import { UseBefore, JsonController, Get, QueryParams } from 'routing-controllers';
+import { Service } from 'typedi';
 import { GetLeaderboardResponse } from '../../types';
 import { UserAuthentication } from '../middleware/UserAuthentication';
 import UserAccountService from '../../services/UserAccountService';
 import { SlidingLeaderboardQueryParams } from '../validators/LeaderboardControllerRequests';
 
 @UseBefore(UserAuthentication)
+@Service()
 @JsonController('/leaderboard')
 export class LeaderboardController {
   private userAccountService: UserAccountService;

api/controllers/MerchStoreController.ts

@@ -13,6 +13,7 @@ import {
   UploadedFile,
 } from 'routing-controllers';
 import { v4 as uuid } from 'uuid';
+import { Service } from 'typedi';
 import PermissionsService from '../../services/PermissionsService';
 import { UserAuthentication } from '../middleware/UserAuthentication';
 import {
@@ -76,6 +77,7 @@ import { UserError } from '../../utils/Errors';
 import StorageService from '../../services/StorageService';
 
 @UseBefore(UserAuthentication)
+@Service()
 @JsonController('/merch')
 export class MerchStoreController {
   private merchStoreService: MerchStoreService;

api/controllers/ResumeController.ts

@@ -12,6 +12,7 @@ import {
   Delete,
 } from 'routing-controllers';
 import * as path from 'path';
+import { Service } from 'typedi';
 import PermissionsService from '../../services/PermissionsService';
 import StorageService from '../../services/StorageService';
 import { UserAuthentication } from '../middleware/UserAuthentication';
@@ -24,6 +25,7 @@ import { PatchResumeRequest, UploadResumeRequest } from '../validators/ResumeCon
 import { UuidParam } from '../validators/GenericRequests';
 
 @UseBefore(UserAuthentication)
+@Service()
 @JsonController('/resume')
 export class ResumeController {
   private resumeService: ResumeService;

api/controllers/UserController.ts

@@ -1,6 +1,7 @@
 import {
   JsonController, Params, Get, Post, Patch, UseBefore, UploadedFile, Body, Delete,
 } from 'routing-controllers';
+import { Service } from 'typedi';
 import { UserModel } from '../../models/UserModel';
 import UserAccountService from '../../services/UserAccountService';
 import UserSocialMediaService from '../../services/UserSocialMediaService';
@@ -27,6 +28,7 @@ import {
 } from '../validators/UserSocialMediaControllerRequests';
 
 @UseBefore(UserAuthentication)
+@Service()
 @JsonController('/user')
 export class UserController {
   private userAccountService: UserAccountService;

api/controllers/index.ts

@@ -1,4 +1,4 @@
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
+ 
 import { UserModel } from '../../models/UserModel';
 import { UserController } from './UserController';
 import { AuthController } from './AuthController';

api/middleware/ErrorHandler.ts

@@ -1,7 +1,9 @@
 import * as express from 'express';
 import { ExpressErrorMiddlewareInterface, Middleware } from 'routing-controllers';
+import { Service } from 'typedi';
 import { handleError } from '../../error';
 
+@Service()
 @Middleware({ type: 'after' })
 export class ErrorHandler implements ExpressErrorMiddlewareInterface {
   error(error: Error, request: express.Request, response: express.Response, next: (err?: any) => any): void {

api/middleware/MetricsRecorder.ts

@@ -1,7 +1,9 @@
 import { ExpressMiddlewareInterface, Middleware } from 'routing-controllers';
 import * as express from 'express';
 import * as metrics from 'datadog-metrics';
+import { Service } from 'typedi';
 
+@Service()
 @Middleware({ type: 'after', priority: 1 })
 export class MetricsRecorder implements ExpressMiddlewareInterface {
   async use(request: express.Request, response: express.Response, next?: express.NextFunction) {

api/middleware/NotFoundHandler.ts

@@ -1,7 +1,9 @@
 import { ExpressMiddlewareInterface, NotFoundError, Middleware } from 'routing-controllers';
 import * as express from 'express';
+import { Service } from 'typedi';
 import { handleError } from '../../error';
 
+@Service()
 @Middleware({ type: 'after' })
 export class NotFoundHandler implements ExpressMiddlewareInterface {
   use(request: express.Request, response: express.Response, next: express.NextFunction) {

api/middleware/RequestLogger.ts

@@ -3,7 +3,9 @@ import * as crypto from 'crypto';
 import * as express from 'express';
 import * as moment from 'moment';
 import * as morgan from 'morgan';
+import { Service } from 'typedi';
 
+@Service()
 @Middleware({ type: 'before' })
 export class RequestLogger implements ExpressMiddlewareInterface {
   async use(request: express.Request, response: express.Response, next?: express.NextFunction) {

api/middleware/UserAuthentication.ts

@@ -1,10 +1,11 @@
 import { ExpressMiddlewareInterface, ForbiddenError } from 'routing-controllers';
 import * as express from 'express';
-import { Inject } from 'typedi';
+import { Inject, Service } from 'typedi';
 import UserAuthService from '../../services/UserAuthService';
 import { authActionMetadata } from '../../utils/AuthActionMetadata';
 import { logger as log } from '../../utils/Logger';
 
+@Service()
 export class UserAuthentication implements ExpressMiddlewareInterface {
   @Inject()
   private userAuthService: UserAuthService;
@@ -28,7 +29,7 @@ export class OptionalUserAuthentication implements ExpressMiddlewareInterface {
       request.user = await this.userAuthService.checkAuthToken(authHeader);
       log.info('user authentication (middleware)', authActionMetadata(request.trace, request.user));
     } catch (error) {
-      log.debug('optional user auth (middleware)');
+      log.debug(`optional user auth (middleware): ${error}`);
     }
     return next();
   }

config/DatabaseNamingStrategy.ts

@@ -1,7 +1,7 @@
 import { DefaultNamingStrategy, NamingStrategyInterface, Table } from 'typeorm';
 
 export class DatabaseNamingStrategy extends DefaultNamingStrategy implements NamingStrategyInterface {
-  private getTableName(tableOrName: string | Table) {
+  protected getTableName(tableOrName: string | Table) {
     const tableName = typeof tableOrName === 'string' ? tableOrName : tableOrName.name;
     return tableName;
   }

eslint.config.js

@@ -0,0 +1,68 @@
+const eslintParser = require('@typescript-eslint/parser');
+
+module.exports = [
+  {
+    // Apply to all files
+    files: ['**/*.ts', '**/*.tsx'],
+    ignores: ['node_modules/**', 'build/**', 'logs/**', '.env'],
+    languageOptions: {
+      parser: eslintParser,
+      parserOptions: {
+        project: './tsconfig.json',
+        ecmaVersion: 2020,
+        sourceType: 'module',
+      },
+      globals: {
+        browser: true,
+        node: true,
+        jest: true,
+      },
+    },
+    plugins: {
+      '@typescript-eslint': require('@typescript-eslint/eslint-plugin'),
+    },
+    rules: {
+      '@typescript-eslint/no-misused-promises': 'error',
+      '@typescript-eslint/no-unused-vars': [
+        'error',
+        {
+          argsIgnorePattern: 'type|_*',
+        },
+      ],
+      'class-methods-use-this': 'off',
+      'consistent-return': 'off',
+      'func-names': 'off',
+      'global-require': 'off',
+      'import/no-cycle': 'off',
+      'import/prefer-default-export': 'off',
+      'max-classes-per-file': 'off',
+      'max-len': [
+        'error',
+        {
+          code: 120,
+        },
+      ],
+      'no-bitwise': [
+        'error',
+        {
+          allow: ['^'],
+        },
+      ],
+      'no-param-reassign': 'off',
+      'no-unused-vars': 'off',
+      'object-curly-newline': [
+        'error',
+        {
+          consistent: true,
+        },
+      ],
+    },
+  },
+  {
+    // Specific rules for test files
+    files: ['tests/*.test.ts'],
+    rules: {
+      'no-await-in-loop': 'off',
+    },
+  },
+];

index.ts

@@ -1,39 +1,21 @@
 import 'reflect-metadata'; // this shim is required
-
-import { createExpressServer, useContainer as routingUseContainer } from 'routing-controllers';
-
-import { createConnection, useContainer as ormUseContainer } from 'typeorm';
-import { Container } from 'typedi';
-import { models as entities } from './models';
-
+import Container from 'typedi';
+import { createExpressServer, useContainer } from 'routing-controllers';
+import { dataSource } from './DataSource';
 import { Config } from './config';
-import { InMemoryDatabaseCache } from './utils/InMemoryDatabaseCache';
-import { logger as log } from './utils/Logger';
 import { controllers } from './api/controllers';
 import { middlewares } from './api/middleware';
 
-routingUseContainer(Container);
-ormUseContainer(Container);
+dataSource
+  .initialize()
+  .then(() => {
+    console.log('created connection');
+  })
+  .catch((error) => {
+    console.log(error);
+  });
 
-createConnection({
-  type: 'postgres',
-  host: Config.database.host,
-  port: Config.database.port,
-  username: Config.database.user,
-  password: Config.database.pass,
-  database: Config.database.name,
-  entities,
-  logging: Config.isDevelopment,
-  cache: {
-    provider(_connection) {
-      return new InMemoryDatabaseCache();
-    },
-  },
-}).then(() => {
-  log.info('created connection');
-}).catch((error) => {
-  log.error(error);
-});
+useContainer(Container);
 
 const app = createExpressServer({
   cors: true,