adminjs experimentation

.gitignore

@@ -36,6 +36,7 @@ lerna-debug.log*
 
 # local development environment files
 .env
+.adminjs
 /devdocker
 /config*.toml
 /private

package.json

@@ -24,6 +24,9 @@
     "test:e2e": "jest --config ./test/jest-e2e.json"
   },
   "dependencies": {
+    "@adminjs/express": "^6.1.0",
+    "@adminjs/nestjs": "^6.1.0",
+    "@adminjs/typeorm": "^5.0.1",
     "@icynet/oauth2-provider": "^1.0.8",
     "@nestjs/cache-manager": "^2.2.1",
     "@nestjs/common": "^10.3.3",
@@ -32,6 +35,7 @@
     "@nestjs/serve-static": "^4.0.1",
     "@nestjs/swagger": "^7.3.0",
     "@nestjs/throttler": "^5.1.2",
+    "adminjs": "^7.7.2",
     "bcrypt": "^5.1.1",
     "cache-manager": "^5.4.0",
     "cache-manager-redis-yet": "^4.1.2",
@@ -43,6 +47,7 @@
     "cropperjs": "^1.6.1",
     "csrf": "^3.1.0",
     "dotenv": "^16.4.4",
+    "express-formidable": "^1.2.0",
     "express-session": "^1.18.0",
     "express-useragent": "^1.0.15",
     "geoip-lite": "^1.4.10",

src/app.module.ts

@@ -13,6 +13,8 @@ import { SSRFrontEndModule } from './modules/ssr-front-end/ssr-front-end.module'
 import { UtilityModule } from './modules/utility/utility.module';
 import { WellKnownModule } from './modules/well-known/well-known.module';
 import { CommonCacheModule } from './modules/cache/cache.module';
+import { AdminjsModule } from './modules/adminjs/adminjs.module';
+import { AdminjsService } from './modules/adminjs/adminjs.service';
 
 @Module({
   imports: [
@@ -37,6 +39,14 @@ import { CommonCacheModule } from './modules/cache/cache.module';
     SSRFrontEndModule,
     WellKnownModule,
     ApiModule,
+    // TODO: https://docs.adminjs.co/installation/plugins/nest
+    import('@adminjs/nestjs').then(({ AdminModule }) =>
+      AdminModule.createAdminAsync({
+        imports: [AdminjsModule],
+        useFactory: (shims: AdminjsService) => shims.getConfiguration(),
+        inject: [AdminjsService],
+      }),
+    ),
   ],
   controllers: [AppController],
   providers: [AppService, CSRFMiddleware],

src/guards/login-antispam.guard.ts

@@ -0,0 +1,63 @@
+import {
+  Injectable,
+  CanActivate,
+  ExecutionContext,
+  HttpException,
+} from '@nestjs/common';
+import { Request } from 'express';
+import { IPLimitService } from 'src/modules/iplimit/iplimit.service';
+import { AuditAction } from 'src/modules/objects/audit/audit.enum';
+import { AuditService } from 'src/modules/objects/audit/audit.service';
+
+@Injectable()
+export class LoginAntispamGuard implements CanActivate {
+  constructor(
+    private iplimit: IPLimitService,
+    private audit: AuditService,
+  ) {}
+
+  async canActivate(context: ExecutionContext): Promise<boolean> {
+    const request = context.switchToHttp().getRequest<Request>();
+
+    if (['GET', 'OPTIONS'].includes(request.method)) return true;
+
+    const known = await this.iplimit.getAddressLimit(request.ip);
+    if (known && known.attempts > 3) {
+      if (known.attempts > 5) {
+        let reported = false;
+        if (!known.reported) {
+          reported = true;
+          await this.audit.insertAudit(
+            AuditAction.THROTTLE,
+            `antispam-guard ${known.attempts} attempts`,
+            undefined,
+            request.ip,
+            request.header('user-agent'),
+          );
+        }
+
+        const limitMinutes = known.attempts > 10 ? 30 : 10; // Half-Hour
+        await this.iplimit.limitUntil(
+          request.ip,
+          limitMinutes * 60 * 1000,
+          reported,
+        );
+
+        await new Promise((resolve) =>
+          setTimeout(resolve, known.attempts * 1000),
+        );
+
+        throw new HttpException(
+          `Too Many Requests. Try again in ${limitMinutes} minutes.`,
+          429,
+        );
+      }
+
+      await new Promise((resolve) => setTimeout(resolve, 1000));
+    }
+
+    await this.iplimit.limitUntil(request.ip, 30 * 1000); // 30 seconds
+
+    return true;
+  }
+}

src/main.ts

@@ -1,7 +1,7 @@
 import { NestFactory } from '@nestjs/core';
 import { AppModule } from './app.module';
-import * as dotenv from 'dotenv';
+import dotenv from 'dotenv';
-import * as cookieParser from 'cookie-parser';
+import cookieParser from 'cookie-parser';
 import { join } from 'path';
 import { SwaggerModule, DocumentBuilder } from '@nestjs/swagger';
 import { NestExpressApplication } from '@nestjs/platform-express';
@@ -9,7 +9,6 @@ import { AdminApiModule } from './modules/api/admin/admin.module';
 import { OAuth2RouterModule } from './modules/ssr-front-end/oauth2-router/oauth2-router.module';
 import { ConfigurationService } from './modules/config/config.service';
 import { ApiModule } from './modules/api/api.module';
-import { AccountApiModule } from './modules/api/account/account.module';
 
 dotenv.config();
 
@@ -45,7 +44,7 @@ async function bootstrap() {
     .build();
 
   const document = SwaggerModule.createDocument(app, docBuilder, {
-    include: [ApiModule, AdminApiModule, OAuth2RouterModule, AccountApiModule],
+    include: [ApiModule, AdminApiModule, OAuth2RouterModule],
   });
 
   SwaggerModule.setup('api/openapi', app, document);

src/modules/adminjs/adminjs.module.ts

@@ -0,0 +1,12 @@
+import { Module } from '@nestjs/common';
+import { ConfigurationModule } from '../config/config.module';
+import { AdminjsService } from './adminjs.service';
+import { ObjectsModule } from '../objects/objects.module';
+import { UserResource } from './resources/user.resource';
+
+@Module({
+  imports: [ConfigurationModule, ObjectsModule],
+  providers: [AdminjsService, UserResource],
+  exports: [AdminjsService],
+})
+export class AdminjsModule {}

src/modules/adminjs/adminjs.service.ts

@@ -0,0 +1,46 @@
+import { Injectable } from '@nestjs/common';
+import { ConfigurationService } from '../config/config.service';
+import { Upload } from '../objects/upload/upload.entity';
+import { Privilege } from '../objects/privilege/privilege.entity';
+import { OAuth2Client } from '../objects/oauth2-client/oauth2-client.entity';
+import { FormUtilityService } from '../utility/services/form-utility.service';
+import { TokenService } from '../utility/services/token.service';
+import { UserResource } from './resources/user.resource';
+
+/**
+ * Shim service for AdminJs features
+ */
+@Injectable({})
+export class AdminjsService {
+  constructor(
+    private readonly config: ConfigurationService,
+    private readonly formUtil: FormUtilityService,
+    private readonly token: TokenService,
+    private readonly userResource: UserResource,
+  ) {}
+
+  async getConfiguration() {
+    const { AdminJS } = await import('adminjs');
+    const AdminJSTypeORM = await import('@adminjs/typeorm');
+    AdminJS.registerAdapter({
+      Database: AdminJSTypeORM.Database,
+      Resource: AdminJSTypeORM.Resource,
+    });
+    return {
+      adminJsOptions: {
+        rootPath: '/admin',
+        resources: [this.userResource, Upload, Privilege, OAuth2Client],
+      },
+      // auth: {
+      //   authenticate: (email, password) => Promise.resolve({ email }),
+      //   cookieName: 'adminjs',
+      //   cookiePassword: 'secret',
+      // },
+      sessionOptions: {
+        resave: true,
+        saveUninitialized: true,
+        secret: 'secret',
+      },
+    };
+  }
+}

src/modules/adminjs/resources/user.resource.ts

@@ -0,0 +1,18 @@
+import { Injectable } from '@nestjs/common';
+import { User } from 'src/modules/objects/user/user.entity';
+
+@Injectable()
+export class UserResource {
+  public resource = User;
+  public options = {
+    listProperties: [
+      'id',
+      'uuid',
+      'email',
+      'username',
+      'display_name',
+      'activated',
+      'created_at',
+    ],
+  };
+}

src/modules/api/account/account.module.ts

@@ -1,34 +0,0 @@
-import { MiddlewareConsumer, Module, NestModule } from '@nestjs/common';
-import * as cors from 'cors';
-import { ConfigurationModule } from 'src/modules/config/config.module';
-import { ConfigurationService } from 'src/modules/config/config.service';
-import { JWTModule } from 'src/modules/jwt/jwt.module';
-import { OAuth2Module } from 'src/modules/oauth2/oauth2.module';
-import { ObjectsModule } from 'src/modules/objects/objects.module';
-import { AccountApiController } from './controllers/account.controller';
-import { AccountApiService } from './services/account.service';
-import { UserTokenModule } from 'src/modules/objects/user-token/user-token.module';
-
-@Module({
-  controllers: [AccountApiController],
-  imports: [
-    ConfigurationModule,
-    JWTModule,
-    ObjectsModule,
-    UserTokenModule,
-    OAuth2Module,
-    AccountApiModule,
-  ],
-  providers: [AccountApiService],
-})
-export class AccountApiModule implements NestModule {
-  constructor(private readonly config: ConfigurationService) {}
-
-  configure(consumer: MiddlewareConsumer) {
-    const corsOpts = cors({
-      origin: [this.config.get('app.base_url'), this.config.get('app.fe_url')],
-      credentials: true,
-    });
-    consumer.apply(corsOpts).forRoutes(AccountApiController);
-  }
-}

src/modules/api/account/controllers/account.controller.ts

@@ -1,45 +0,0 @@
-import {
-  ClassSerializerInterceptor,
-  Controller,
-  Get,
-  UseGuards,
-  UseInterceptors,
-  UsePipes,
-  ValidationPipe,
-} from '@nestjs/common';
-import {
-  ApiBearerAuth,
-  ApiTags,
-  ApiOAuth2,
-  ApiOperation,
-  ApiOkResponse,
-} from '@nestjs/swagger';
-import { Scopes } from 'src/decorators/scopes.decorator';
-import { OAuth2Guard } from 'src/guards/oauth2.guard';
-import { PrivilegesGuard } from 'src/guards/privileges.guard';
-import { ScopesGuard } from 'src/guards/scopes.guard';
-import { AccountApiService } from '../services/account.service';
-import { CurrentUser } from 'src/decorators/user.decorator';
-import { User } from 'src/modules/objects/user/user.entity';
-import { AccountResponseDto } from '../../dtos/account.dto';
-
-@Controller({
-  path: '/api/account',
-})
-@ApiBearerAuth()
-@ApiTags('account')
-@ApiOAuth2(['account'])
-@Scopes('account')
-@UseInterceptors(ClassSerializerInterceptor)
-@UsePipes(new ValidationPipe({ whitelist: true }))
-@UseGuards(OAuth2Guard, PrivilegesGuard, ScopesGuard)
-export class AccountApiController {
-  constructor(private readonly service: AccountApiService) {}
-
-  @Get()
-  @ApiOperation({ summary: 'Get account details' })
-  @ApiOkResponse({ type: AccountResponseDto })
-  async getAccount(@CurrentUser() user: User) {
-    return this.service.getAccountInfo(user);
-  }
-}

src/modules/api/account/services/account.service.ts

@@ -1,29 +0,0 @@
-import { Injectable } from '@nestjs/common';
-import { plainToInstance } from 'class-transformer';
-import { UserTOTPService } from 'src/modules/objects/user-token/user-totp-token.service';
-import { User } from 'src/modules/objects/user/user.entity';
-import { UserService } from 'src/modules/objects/user/user.service';
-import { FormUtilityService } from 'src/modules/utility/services/form-utility.service';
-import { AccountResponseDto } from '../../dtos/account.dto';
-
-@Injectable()
-export class AccountApiService {
-  constructor(
-    private readonly userService: UserService,
-    private readonly userTotp: UserTOTPService,
-    private readonly form: FormUtilityService,
-  ) {}
-
-  async getAccountInfo(user: User) {
-    const fullInfo = await this.userService.getByUUID(user.uuid, [
-      'picture',
-      'privileges',
-    ]);
-    const hasTotp = await this.userTotp.userHasTOTP(fullInfo);
-
-    return plainToInstance(AccountResponseDto, {
-      ...fullInfo,
-      totp_enabled: hasTotp,
-    });
-  }
-}

src/modules/api/admin/admin.module.ts

@@ -5,12 +5,12 @@ import * as mime from 'mime-types';
 import { join } from 'path';
 import { OAuth2Module } from 'src/modules/oauth2/oauth2.module';
 import { ObjectsModule } from 'src/modules/objects/objects.module';
-import { OAuth2AdminController } from './controllers/oauth2-admin.controller';
+import { OAuth2AdminController } from './oauth2-admin.controller';
-import { PrivilegeAdminController } from './controllers/privilege-admin.controller';
+import { PrivilegeAdminController } from './privilege-admin.controller';
-import { UserAdminController } from './controllers/user-admin.controller';
+import { UserAdminController } from './user-admin.controller';
 import { ConfigurationModule } from 'src/modules/config/config.module';
-import { AdminService } from './services/admin.service';
+import { AdminService } from './admin.service';
-import { AuditAdminController } from './controllers/audit-admin.controller';
+import { AuditAdminController } from './audit-admin.controller';
 
 @Module({
   controllers: [

src/modules/api/admin/admin.service.ts

@@ -2,7 +2,7 @@ import { Injectable } from '@nestjs/common';
 import { OAuth2Client } from 'src/modules/objects/oauth2-client/oauth2-client.entity';
 import { User } from 'src/modules/objects/user/user.entity';
 
-const UNPRIVILEGED_STRIP = ['id_token', 'management', 'implicit', 'account'];
+const UNPRIVILEGED_STRIP = ['id_token', 'management', 'implicit'];
 
 @Injectable()
 export class AdminService {

src/modules/api/admin/oauth2-admin.controller.ts

@@ -36,7 +36,7 @@ import { FormUtilityService } from 'src/modules/utility/services/form-utility.se
 import { PaginationService } from 'src/modules/utility/services/paginate.service';
 import { TokenService } from 'src/modules/utility/services/token.service';
 import { PageOptions } from 'src/types/pagination.interfaces';
-import { AdminService } from '../services/admin.service';
+import { AdminService } from './admin.service';
 import { OAuth2TokenService } from 'src/modules/objects/oauth2-token/oauth2-token.service';
 import { Throttle, ThrottlerGuard } from '@nestjs/throttler';
 

src/modules/api/admin/user-admin.controller.ts

@@ -67,19 +67,6 @@ export class UserAdminController {
     };
   }
 
-  /**
-   * Create a registraion invitation and send email.
-   * @param body
-   * @returns Success
-   */
-  @Post('invite')
-  @Scopes('management')
-  @Privileges('admin', 'admin:user')
-  async invite(@Body() { email }: Pick<User, 'email'>) {
-    await this._user.issueRegistrationToken(email);
-    return { success: true };
-  }
-
   /**
    * Get a single user by ID
    * @param id User ID

src/modules/api/api.module.ts

@@ -1,5 +1,5 @@
 import { MiddlewareConsumer, Module, NestModule } from '@nestjs/common';
-import * as cors from 'cors';
+import cors from 'cors';
 import { ConfigurationModule } from '../config/config.module';
 import { JWTModule } from '../jwt/jwt.module';
 import { OAuth2Module } from '../oauth2/oauth2.module';
@@ -7,7 +7,6 @@ import { OAuth2Service } from '../oauth2/oauth2.service';
 import { ObjectsModule } from '../objects/objects.module';
 import { AdminApiModule } from './admin/admin.module';
 import { ApiController } from './api.controller';
-import { AccountApiModule } from './account/account.module';
 
 @Module({
   controllers: [ApiController],
@@ -15,9 +14,8 @@ import { AccountApiModule } from './account/account.module';
     ConfigurationModule,
     JWTModule,
     ObjectsModule,
-    OAuth2Module,
     AdminApiModule,
-    AccountApiModule,
+    OAuth2Module,
   ],
 })
 export class ApiModule implements NestModule {

src/modules/api/dtos/account.dto.ts

@@ -1,7 +0,0 @@
-import { ApiProperty, OmitType } from '@nestjs/swagger';
-import { User } from 'src/modules/objects/user/user.entity';
-
-export class AccountResponseDto extends OmitType(User, ['password']) {
-  @ApiProperty()
-  totp_enabled: boolean;
-}

src/modules/config/config.providers.ts

@@ -17,7 +17,6 @@ export const configProviders = [
     useValue: {
       app: {
         base_url: 'http://localhost:3000',
-        fe_url: 'http://localhost:5173',
         host: '0.0.0.0',
         port: 3000,
         session_name: '__sid',

src/modules/iplimit/iplimit.module.ts

@@ -0,0 +1,10 @@
+import { Module } from '@nestjs/common';
+import { IPLimitService } from './iplimit.service';
+import { CommonCacheModule } from '../cache/cache.module';
+
+@Module({
+  imports: [CommonCacheModule],
+  providers: [IPLimitService],
+  exports: [IPLimitService],
+})
+export class IPLimitModule {}

src/modules/iplimit/iplimit.service.ts

@@ -0,0 +1,47 @@
+import { CACHE_MANAGER } from '@nestjs/cache-manager';
+import { Inject, Injectable } from '@nestjs/common';
+import { Cache } from 'cache-manager';
+import { TokenService } from '../utility/services/token.service';
+
+export interface IPLimit {
+  ip: string;
+  attempts: number;
+  reported: boolean;
+}
+
+@Injectable()
+export class IPLimitService {
+  constructor(
+    @Inject(CACHE_MANAGER)
+    private readonly cache: Cache,
+    private readonly token: TokenService,
+  ) {}
+
+  public async getAddressLimit(ip: string) {
+    const ipHash = this.token.insecureHash(ip);
+    const entry = await this.cache.get<IPLimit>(`iplimit-${ipHash}`);
+    if (!entry) return null;
+    return entry;
+  }
+
+  public async limitUntil(ip: string, expires: number, reported = false) {
+    const ipHash = this.token.insecureHash(ip);
+    const existing = await this.cache.get<IPLimit>(`iplimit-${ipHash}`);
+    if (existing) {
+      existing.attempts++;
+      if (reported) existing.reported = true;
+      await this.cache.set(`iplimit-${ipHash}`, existing, expires + Date.now());
+      return existing;
+    }
+
+    const newObj = {
+      ip,
+      attempts: 0,
+      reported,
+    };
+
+    await this.cache.set(`iplimit-${ipHash}`, newObj, expires + Date.now());
+
+    return newObj;
+  }
+}

src/modules/oauth2/adapter/user.adapter.ts

@@ -6,6 +6,7 @@ import { ParsedQs } from 'qs';
 import { OAuth2ClientService } from 'src/modules/objects/oauth2-client/oauth2-client.service';
 import { UserService } from 'src/modules/objects/user/user.service';
 import { FormUtilityService } from 'src/modules/utility/services/form-utility.service';
+import { TokenService } from 'src/modules/utility/services/token.service';
 
 @Injectable()
 export class UserAdapter implements OAuth2UserAdapter {
@@ -13,6 +14,7 @@ export class UserAdapter implements OAuth2UserAdapter {
     private readonly userService: UserService,
     private readonly clientService: OAuth2ClientService,
     private readonly form: FormUtilityService,
+    private readonly token: TokenService,
   ) {}
 
   getId(user: OAuth2User): number {
@@ -48,7 +50,7 @@ export class UserAdapter implements OAuth2UserAdapter {
   }
 
   checkPassword(user: OAuth2User, password: string): Promise<boolean> {
-    return this.userService.comparePasswords(user.password, password);
+    return this.token.comparePasswords(user.password, password);
   }
 
   async fetchFromRequest(

src/modules/oauth2/oauth2.service.ts

@@ -11,7 +11,6 @@ import { UserAdapter } from './adapter/user.adapter';
 const SCOPE_DESCRIPTION: Record<string, string> = {
   email: 'Email address',
   picture: 'Profile picture',
-  account: 'Password and other account settings',
 };
 
 const ALWAYS_AVAILABLE = ['Username and display name'];
@@ -40,10 +39,6 @@ export class OAuth2Service implements OAuth2AdapterModel {
       disallowedScopes = null;
     }
 
-    if (scope.includes('account')) {
-      disallowedScopes = null;
-    }
-
     res.render('authorize', {
       csrf: req.csrfToken(),
       user: req.user,

src/modules/objects/audit/audit.entity.ts

@@ -1,4 +1,5 @@
 import {
+  BaseEntity,
   Column,
   CreateDateColumn,
   Entity,
@@ -8,7 +9,7 @@ import {
 import { User } from '../user/user.entity';
 
 @Entity()
-export class AuditLog {
+export class AuditLog extends BaseEntity {
   @PrimaryGeneratedColumn()
   id: number;
 

src/modules/objects/audit/audit.interfaces.ts

@@ -1,6 +1,7 @@
 import { Lookup } from 'geoip-lite';
 import { Details } from 'express-useragent';
 import { AuditAction } from './audit.enum';
+import { AuditLog } from './audit.entity';
 
 export interface UserLoginEntry {
   login_at: Date;
@@ -17,3 +18,8 @@ export interface AuditSearchClause {
   content?: string;
   flagged?: boolean;
 }
+
+export interface AuditResponse extends AuditLog {
+  location?: Partial<Lookup>;
+  user_agent?: Partial<Details>;
+}

src/modules/objects/audit/audit.service.ts

@@ -10,10 +10,14 @@ import {
 import { User } from '../user/user.entity';
 import { AuditLog } from './audit.entity';
 import { AuditAction } from './audit.enum';
-import { Lookup, lookup } from 'geoip-lite';
+import { lookup } from 'geoip-lite';
-import { Details, parse } from 'express-useragent';
+import { parse } from 'express-useragent';
 import { FormUtilityService } from 'src/modules/utility/services/form-utility.service';
-import { AuditSearchClause, UserLoginEntry } from './audit.interfaces';
+import {
+  AuditResponse,
+  AuditSearchClause,
+  UserLoginEntry,
+} from './audit.interfaces';
 
 const PLUCK_LOCATION = ['country', 'city', 'timezone', 'll'];
 const PLUCK_USER_AGENT = ['browser', 'version', 'os', 'platform'];
@@ -86,15 +90,7 @@ export class AuditService {
     limit = 50,
     offset = 0,
     search: AuditSearchClause,
-  ): Promise<
+  ): Promise<[AuditResponse[], number]> {
-    [
-      (AuditLog & {
-        location?: Partial<Lookup>;
-        user_agent?: Partial<Details>;
-      })[],
-      number,
-    ]
-  > {
     const [list, num] = await this.audit.findAndCount({
       ...this.buildAuditSearch(search),
       take: limit,
@@ -104,22 +100,25 @@ export class AuditService {
     });
 
     return [
-      list.map((entry) => ({
+      list.map(
-        ...entry,
+        (entry) =>
-        location: entry.actor_ip
+          ({
-          ? this.form.pluckObject(
+            ...entry,
-              this.getIPLocation(entry.actor_ip),
+            location: entry.actor_ip
-              PLUCK_LOCATION,
+              ? this.form.pluckObject(
-            )
+                  this.getIPLocation(entry.actor_ip),
-          : null,
+                  PLUCK_LOCATION,
-        user_agent: entry.actor_ua
+                )
-          ? this.form.pluckObject(
+              : null,
-              this.getUserAgentInfo(entry.actor_ua),
+            user_agent: entry.actor_ua
-              PLUCK_USER_AGENT,
+              ? this.form.pluckObject(
-            )
+                  this.getUserAgentInfo(entry.actor_ua),
-          : null,
+                  PLUCK_USER_AGENT,
-        actor: this.form.stripObject(entry.actor, ['password']),
+                )
-      })),
+              : null,
+            actor: this.form.stripObject(entry.actor, ['password']),
+          }) as AuditResponse,
+      ),
       num,
     ];
   }

src/modules/objects/document/document.entity.ts

@@ -1,4 +1,5 @@
 import {
+  BaseEntity,
   Column,
   CreateDateColumn,
   Entity,
@@ -9,7 +10,7 @@ import {
 import { User } from '../user/user.entity';
 
 @Entity()
-export class Document {
+export class Document extends BaseEntity {
   @PrimaryGeneratedColumn()
   id: number;
 

src/modules/objects/document/document.service.ts

@@ -38,7 +38,7 @@ export class DocumentService {
     return {
       ...doc,
       html,
-    };
+    } as Document & { html: string };
   }
 
   public async getDocumentByID(
@@ -49,6 +49,6 @@ export class DocumentService {
     return {
       ...doc,
       html,
-    };
+    } as Document & { html: string };
   }
 }

src/modules/objects/oauth2-client/oauth2-client.entity.ts

@@ -1,4 +1,5 @@
 import {
+  BaseEntity,
   Column,
   CreateDateColumn,
   Entity,
@@ -12,7 +13,7 @@ import { User } from '../user/user.entity';
 import { OAuth2ClientURL } from './oauth2-client-url.entity';
 
 @Entity()
-export class OAuth2Client {
+export class OAuth2Client extends BaseEntity {
   @PrimaryGeneratedColumn()
   id: number;
 

src/modules/objects/oauth2-client/oauth2-client.service.ts

@@ -26,7 +26,6 @@ export class OAuth2ClientService {
     'email',
     'privileges',
     'management',
-    'account',
     'openid',
   ];
 

src/modules/objects/oauth2-token/oauth2-token.entity.ts

@@ -1,4 +1,5 @@
 import {
+  BaseEntity,
   Column,
   CreateDateColumn,
   Entity,
@@ -16,7 +17,7 @@ export enum OAuth2TokenType {
 }
 
 @Entity()
-export class OAuth2Token {
+export class OAuth2Token extends BaseEntity {
   @PrimaryGeneratedColumn()
   id: number;
 

src/modules/objects/privilege/privilege.entity.ts

@@ -1,14 +1,11 @@
-import { ApiProperty } from '@nestjs/swagger';
+import { BaseEntity, Column, Entity, PrimaryGeneratedColumn } from 'typeorm';
-import { Column, Entity, PrimaryGeneratedColumn } from 'typeorm';
 
 @Entity()
-export class Privilege {
+export class Privilege extends BaseEntity {
   @PrimaryGeneratedColumn()
-  @ApiProperty()
   id: number;
 
   @Column({ type: 'text', nullable: false })
-  @ApiProperty()
   name: string;
 }
 

src/modules/objects/upload/upload.entity.ts

@@ -1,4 +1,5 @@
 import {
+  BaseEntity,
   Column,
   CreateDateColumn,
   Entity,
@@ -7,26 +8,19 @@ import {
   UpdateDateColumn,
 } from 'typeorm';
 import { User } from '../user/user.entity';
-import { Exclude, Expose } from 'class-transformer';
-import { ApiProperty } from '@nestjs/swagger';
 
 @Entity()
-@Expose()
+export class Upload extends BaseEntity {
-export class Upload {
   @PrimaryGeneratedColumn()
-  @ApiProperty()
   id: number;
 
   @Column({ nullable: false })
-  @ApiProperty()
   original_name: string;
 
   @Column({ nullable: false })
-  @ApiProperty()
   mimetype: string;
 
   @Column({ nullable: false })
-  @ApiProperty()
   file: string;
 
   @ManyToOne(() => User, {
@@ -34,14 +28,11 @@ export class Upload {
     onDelete: 'SET NULL',
     onUpdate: 'CASCADE',
   })
-  @Exclude()
   uploader: User;
 
   @CreateDateColumn()
-  @ApiProperty()
   public created_at: Date;
 
   @UpdateDateColumn()
-  @ApiProperty()
   public updated_at: Date;
 }

src/modules/objects/user-token/user-token.entity.ts

@@ -1,4 +1,5 @@
 import {
+  BaseEntity,
   Column,
   CreateDateColumn,
   Entity,
@@ -20,7 +21,7 @@ export enum UserTokenType {
 }
 
 @Entity()
-export class UserToken {
+export class UserToken extends BaseEntity {
   @PrimaryGeneratedColumn()
   id: number;
 

src/modules/objects/user/email/invitation.email.ts

@@ -1,22 +0,0 @@
-import { EmailTemplate } from 'src/modules/objects/email/email.template';
-
-export const InvitationEmail = (url: string): EmailTemplate => ({
-  text: `
-Icy Network
-
-Please click on the following link to create an account on Icy Network.
-
-Create your account here: ${url}
-
-This email was sent to you because you have requested an account on Icy Network. If you did not request this, you may safely ignore this email.
-  `,
-  html: /* html */ `
-<h1>Icy Network</h1>
-
-<p><b>Please click on the following link to create an account on Icy Network.</b></p>
-
-<p>Create your account here: <a href="${url}" target="_blank">${url}</a></p>
-
-<p>This email was sent to you because you have requested an account on Icy Network. If you did not request this, you may safely ignore this email.</p>
-  `,
-});

src/modules/objects/user/email/registration.email.ts

@@ -12,8 +12,6 @@ Welcome to Icy Network, ${username}!
 In order to proceed with logging in, please click on the following link to activate your account.
 
 Activate your account: ${url}
-
-This email was sent to you because you have created an account on Icy Network. If you did not create an account, you may contact us or just let the account expire.
   `,
   html: /* html */ `
 <h1>Icy Network</h1>
@@ -23,7 +21,5 @@ This email was sent to you because you have created an account on Icy Network. I
 <p>In order to proceed with logging in, please click on the following link to activate your account.</p>
 
 <p>Activate your account: <a href="${url}" target="_blank">${url}</a></p>
-
-<p>This email was sent to you because you have created an account on Icy Network. If you did not create an account, you may contact us or just let the account expire.</p>
   `,
 });

src/modules/objects/user/user.entity.ts

@@ -1,4 +1,5 @@
 import {
+  BaseEntity,
   Column,
   CreateDateColumn,
   Entity,
@@ -10,53 +11,39 @@ import {
 } from 'typeorm';
 import { Privilege } from '../privilege/privilege.entity';
 import { Upload } from '../upload/upload.entity';
-import { ApiProperty } from '@nestjs/swagger';
-import { Exclude, Expose } from 'class-transformer';
 
 @Entity()
-@Expose()
+export class User extends BaseEntity {
-export class User {
   @PrimaryGeneratedColumn()
-  @ApiProperty()
   id: number;
 
   @Column({ type: 'uuid', length: 36, nullable: false, unique: true })
-  @ApiProperty()
   uuid: string;
 
   @Column({ length: 26, nullable: false, unique: true })
-  @ApiProperty()
   username: string;
 
   @Column({ nullable: false, unique: true })
-  @ApiProperty()
   email: string;
 
   @Column({ length: 32, nullable: false })
-  @ApiProperty()
   display_name: string;
 
   @Column({ type: 'text', nullable: true })
-  @Exclude()
   password: string;
 
   @Column({ default: false })
-  @ApiProperty()
   activated: boolean;
 
   @Column({ type: 'timestamp' })
-  @ApiProperty()
   public activity_at: Date;
 
   @CreateDateColumn()
-  @ApiProperty()
   public created_at: Date;
 
   @UpdateDateColumn()
-  @ApiProperty()
   public updated_at: Date;
 
-  @ApiProperty({ type: () => Upload })
   @ManyToOne(() => Upload, {
     nullable: true,
     onDelete: 'SET NULL',
@@ -64,7 +51,6 @@ export class User {
   })
   public picture: Upload;
 
-  @ApiProperty({ type: Privilege, isArray: true })
   @ManyToMany(() => Privilege)
   @JoinTable()
   public privileges: Privilege[];

src/modules/objects/user/user.module.ts

@@ -6,11 +6,9 @@ import { UploadModule } from '../upload/upload.module';
 import { UserTokenModule } from '../user-token/user-token.module';
 import { userProviders } from './user.providers';
 import { UserService } from './user.service';
-import { CommonCacheModule } from 'src/modules/cache/cache.module';
 
 @Module({
   imports: [
-    CommonCacheModule,
     DatabaseModule,
     EmailModule,
     UserTokenModule,

src/modules/objects/user/user.service.ts

@@ -1,11 +1,5 @@
-import {
+import { Inject, Injectable } from '@nestjs/common';
-  BadRequestException,
-  Inject,
-  Injectable,
-  Logger,
-} from '@nestjs/common';
 import { ILike, Repository } from 'typeorm';
-import * as bcrypt from 'bcrypt';
 
 import { UserTokenType } from '../user-token/user-token.entity';
 import { User } from './user.entity';
@@ -17,14 +11,9 @@ import { UserTokenService } from '../user-token/user-token.service';
 import { ConfigurationService } from 'src/modules/config/config.service';
 import { Upload } from '../upload/upload.entity';
 import { UploadService } from '../upload/upload.service';
-import { CACHE_MANAGER } from '@nestjs/cache-manager';
-import { Cache } from 'cache-manager';
-import { InvitationEmail } from './email/invitation.email';
 
 @Injectable()
 export class UserService {
-  private readonly logger = new Logger(UserService.name);
-
   constructor(
     @Inject('USER_REPOSITORY')
     private userRepository: Repository<User>,
@@ -33,7 +22,6 @@ export class UserService {
     private email: EmailService,
     private config: ConfigurationService,
     private upload: UploadService,
-    @Inject(CACHE_MANAGER) private readonly cache: Cache,
   ) {}
 
   public async getById(id: number, relations?: string[]): Promise<User> {
@@ -151,36 +139,19 @@ export class UserService {
 
     user.picture = upload;
     await this.updateUser(user);
-    this.logger.log(`User ID: ${user.id} avatar has been updated`);
     return user;
   }
 
   public async deleteAvatar(user: User): Promise<void> {
     if (user.picture) {
       await this.upload.delete(user.picture);
-      this.logger.log(`User ID: ${user.id} avatar has been deleted`);
     }
   }
 
-  public async comparePasswords(
-    hash: string,
-    password: string,
-  ): Promise<boolean> {
-    return bcrypt.compare(password, hash);
-  }
-
-  public async hashPassword(password: string): Promise<string> {
-    const salt = await bcrypt.genSalt(10);
-    return bcrypt.hash(password, salt);
-  }
-
   public async sendActivationEmail(
     user: User,
     redirectTo?: string,
   ): Promise<void> {
-    this.logger.log(
-      `Sending an activation email to User ID: ${user.id} (${user.email})`,
-    );
     const activationToken = await this.userToken.create(
       user,
       UserTokenType.ACTIVATION,
@@ -204,22 +175,13 @@ export class UserService {
         'Activate your account on Icy Network',
         content,
       );
-      this.logger.log(
-        `Sending an activation email to User ID: ${user.id} (${user.email}) was successful.`,
-      );
     } catch (e) {
-      this.logger.error(
-        `Sending an activation email to User ID: ${user.id} (${user.email}) failed`,
-      );
       await this.userToken.delete(activationToken);
       throw e;
     }
   }
 
   public async sendPasswordEmail(user: User): Promise<void> {
-    this.logger.log(
-      `Sending a password reset email to User ID: ${user.id} (${user.email})`,
-    );
     const passwordToken = await this.userToken.create(
       user,
       UserTokenType.PASSWORD,
@@ -238,13 +200,7 @@ export class UserService {
         'Reset your password on Icy Network',
         content,
       );
-      this.logger.log(
-        `Sent a password reset email to User ID: ${user.id} (${user.email}) successfully`,
-      );
     } catch (e) {
-      this.logger.error(
-        `Sending a password reset email to User ID: ${user.id} (${user.email}) failed: ${e}`,
-      );
       await this.userToken.delete(passwordToken);
       // silently fail
     }
@@ -267,12 +223,7 @@ export class UserService {
       password: string;
     },
     redirectTo?: string,
-    activate = false,
   ): Promise<User> {
-    this.logger.log(
-      `Starting registration of new user ${newUserInfo.username} (${newUserInfo.email})`,
-    );
-
     if (!!(await this.getByEmail(newUserInfo.email))) {
       throw new Error('Email is already in use!');
     }
@@ -281,7 +232,7 @@ export class UserService {
       throw new Error('Username is already in use!');
     }
 
-    const hashword = await this.hashPassword(newUserInfo.password);
+    const hashword = await this.token.hashPassword(newUserInfo.password);
     const user = new User();
     user.email = newUserInfo.email;
     user.uuid = this.token.createUUID();
@@ -289,71 +240,18 @@ export class UserService {
     user.display_name = newUserInfo.display_name;
     user.password = hashword;
     user.activity_at = new Date();
-    user.activated = activate;
 
-    await this.userRepository.save(user);
+    await this.userRepository.insert(user);
 
-    if (!user.activated) {
+    try {
-      try {
+      await this.sendActivationEmail(user, redirectTo);
-        await this.sendActivationEmail(user, redirectTo);
+    } catch (e) {
-      } catch (e) {
+      await this.userRepository.remove(user);
-        await this.userRepository.remove(user);
+      throw new Error(
-        throw new Error(
+        'Failed to send activation email! Please check your email address and try again!',
-          'Failed to send activation email! Please check your email address and try again!',
+      );
-        );
-      }
     }
 
-    this.logger.log(
-      `Registered a new user ${newUserInfo.username} (${newUserInfo.email}) ID: ${user.id}`,
-    );
-
     return user;
   }
-
-  public async issueRegistrationToken(email: string) {
-    this.logger.log(`Issuing a new registration token for ${email}`);
-
-    const existingUser = await this.getByEmail(email);
-    if (existingUser) {
-      throw new BadRequestException('User by email already exists');
-    }
-
-    const newToken = this.token.generateString(64);
-    await this.cache.set(
-      `register-${newToken}`,
-      email,
-      7 * 24 * 60 * 60 * 1000, // 7 days
-    );
-
-    try {
-      const content = InvitationEmail(
-        `${this.config.get<string>('app.base_url')}/register?token=${newToken}`,
-      );
-
-      await this.email.sendEmailTemplate(
-        email,
-        'You have been invited to create an account on Icy Network',
-        content,
-      );
-
-      this.logger.log(
-        `Issuing a new registration token for ${email} was successful`,
-      );
-    } catch (error) {
-      this.logger.error(
-        `Issuing a new registration token for ${email} failed: ${error}`,
-      );
-      await this.cache.del(`register-${newToken}`);
-      throw error;
-    }
-  }
-
-  public async checkRegistrationToken(token: string) {
-    return await this.cache.get<string>(`register-${token}`);
-  }
-
-  public async invalidateRegistrationToken(token: string) {
-    return await this.cache.del(`register-${token}`);
-  }
 }

src/modules/ssr-front-end/login/login.controller.ts

@@ -13,6 +13,7 @@ import {
 import { Throttle, ThrottlerGuard } from '@nestjs/throttler';
 import { Request, Response } from 'express';
 import { SessionData } from 'express-session';
+import { LoginAntispamGuard } from 'src/guards/login-antispam.guard';
 import { AuditAction } from 'src/modules/objects/audit/audit.enum';
 import { AuditService } from 'src/modules/objects/audit/audit.service';
 import {
@@ -34,6 +35,7 @@ interface VerifyChallenge {
 }
 
 @Controller('/login')
+@UseGuards(LoginAntispamGuard)
 export class LoginController {
   constructor(
     private readonly userService: UserService,
@@ -58,8 +60,6 @@ export class LoginController {
   }
 
   @Post()
-  @Throttle({ default: { limit: 5, ttl: 60000 } })
-  @UseGuards(ThrottlerGuard)
   public async loginRequest(
     @Req() req: Request,
     @Res() res: Response,
@@ -75,7 +75,7 @@ export class LoginController {
     if (
       !user ||
       !user.activated ||
-      !(await this.userService.comparePasswords(user.password, password))
+      !(await this.token.comparePasswords(user.password, password))
     ) {
       req.flash('form', { username });
       req.flash('message', {
@@ -365,7 +365,7 @@ export class LoginController {
         throw new Error('The passwords do not match!');
       }
 
-      const hashword = await this.userService.hashPassword(password);
+      const hashword = await this.token.hashPassword(password);
       token.user.password = hashword;
 
       await this.userService.updateUser(token.user);

src/modules/ssr-front-end/login/login.module.ts

@@ -9,9 +9,16 @@ import { UserTokenModule } from 'src/modules/objects/user-token/user-token.modul
 import { UserModule } from 'src/modules/objects/user/user.module';
 import { SessionModule } from '../session/session.module';
 import { LoginController } from './login.controller';
+import { IPLimitModule } from 'src/modules/iplimit/iplimit.module';
 
 @Module({
-  imports: [UserModule, UserTokenModule, AuditModule, SessionModule],
+  imports: [
+    UserModule,
+    UserTokenModule,
+    AuditModule,
+    SessionModule,
+    IPLimitModule,
+  ],
   controllers: [LoginController],
 })
 export class LoginModule implements NestModule {

src/modules/ssr-front-end/register/register.controller.ts

@@ -4,6 +4,7 @@ import {
   Get,
   Post,
   Query,
+  Render,
   Req,
   Res,
   UnauthorizedException,
@@ -11,6 +12,7 @@ import {
 } from '@nestjs/common';
 import { Throttle, ThrottlerGuard } from '@nestjs/throttler';
 import { Request, Response } from 'express';
+import { LoginAntispamGuard } from 'src/guards/login-antispam.guard';
 import { ConfigurationService } from 'src/modules/config/config.service';
 import { AuditAction } from 'src/modules/objects/audit/audit.enum';
 import { AuditService } from 'src/modules/objects/audit/audit.service';
@@ -19,6 +21,7 @@ import { FormUtilityService } from 'src/modules/utility/services/form-utility.se
 import { RegisterDto } from './register.interfaces';
 
 @Controller('/register')
+@UseGuards(LoginAntispamGuard)
 export class RegisterController {
   constructor(
     private readonly userService: UserService,
@@ -28,77 +31,26 @@ export class RegisterController {
   ) {}
 
   @Get()
-  public async registerView(
+  @Render('register')
-    @Req() req: Request,
+  public registerView(@Req() req: Request): Record<string, unknown> {
-    @Res() res: Response,
+    return this.formUtil.populateTemplate(req, {
-    @Query('token') registrationToken: string,
+      registrationAuthorized: this.config.get<boolean>('app.registrations'),
-    @Query('redirectTo') redirectTo: string,
+    });
-  ) {
-    let registrationAuthorized = this.config.get<boolean>('app.registrations');
-    if (registrationToken) {
-      const registrationEmail =
-        await this.userService.checkRegistrationToken(registrationToken);
-
-      if (!registrationEmail) {
-        req.flash('message', {
-          error: true,
-          text: `This registration token is invalid or expired.`,
-        });
-
-        return res.redirect(
-          '/login' + (redirectTo ? '?redirectTo=' + redirectTo : ''),
-        );
-      }
-
-      req.flash('form', { email: registrationEmail });
-
-      // bypass limitations
-      registrationAuthorized = true;
-    }
-
-    res.render(
-      'register',
-      this.formUtil.populateTemplate(req, {
-        registrationAuthorized,
-      }),
-    );
   }
 
   @Post()
-  @Throttle({ default: { limit: 3, ttl: 60000 } })
+  @Throttle({ default: { limit: 3, ttl: 10000 } })
   @UseGuards(ThrottlerGuard)
   public async registerRequest(
     @Req() req: Request,
     @Res() res: Response,
     @Body() body: RegisterDto,
-    @Query('token') registrationToken: string,
     @Query('redirectTo') redirectTo?: string,
   ) {
     const { username, display_name, email, password, password_repeat } =
       this.formUtil.trimmed(body, ['username', 'display_name', 'email']);
 
-    let registrationAuthorized = this.config.get<boolean>('app.registrations');
+    if (!this.config.get<boolean>('app.registrations')) {
-    let tokenEmail: string | undefined;
-
-    if (registrationToken) {
-      tokenEmail =
-        await this.userService.checkRegistrationToken(registrationToken);
-
-      if (!tokenEmail) {
-        req.flash('message', {
-          error: true,
-          text: `This registration token is invalid or expired.`,
-        });
-        return res.redirect(
-          '/login' + (redirectTo ? '?redirectTo=' + redirectTo : ''),
-        );
-      }
-
-      // bypass limitations
-      registrationAuthorized = true;
-    }
-
-    if (!registrationAuthorized) {
       throw new UnauthorizedException(
         'Registrations are disabled by administrator.',
       );
@@ -142,23 +94,12 @@ export class RegisterController {
         throw new Error('The passwords do not match!');
       }
 
-      const sendActivationEmail = tokenEmail ? tokenEmail !== email : true;
+      const user = await this.userService.userRegistration(body, redirectTo);
-      const user = await this.userService.userRegistration(
-        body,
-        redirectTo,
-        !sendActivationEmail,
-      );
       await this.audit.auditRequest(req, AuditAction.REGISTRATION, null, user);
 
-      if (tokenEmail) {
-        await this.userService.invalidateRegistrationToken(registrationToken);
-      }
-
       req.flash('message', {
         error: false,
-        text: sendActivationEmail
+        text: `An activation email has been sent to ${email}!`,
-          ? `An activation email has been sent to ${email}!`
-          : `Welcome, we have been expecting you! You may now log in.`,
       });
 
       res.redirect('/login' + (redirectTo ? '?redirectTo=' + redirectTo : ''));

src/modules/ssr-front-end/register/register.module.ts

@@ -8,9 +8,10 @@ import { AuditModule } from 'src/modules/objects/audit/audit.module';
 import { UserModule } from 'src/modules/objects/user/user.module';
 import { SessionModule } from '../session/session.module';
 import { RegisterController } from './register.controller';
+import { IPLimitModule } from 'src/modules/iplimit/iplimit.module';
 
 @Module({
-  imports: [UserModule, AuditModule, SessionModule],
+  imports: [UserModule, AuditModule, SessionModule, IPLimitModule],
   controllers: [RegisterController],
 })
 export class RegisterModule implements NestModule {

src/modules/ssr-front-end/session/session.providers.ts

@@ -1,7 +1,7 @@
 import { FactoryProvider } from '@nestjs/common';
 import { ConfigurationService } from 'src/modules/config/config.service';
 
-import * as session from 'express-session';
+import session from 'express-session';
 import RedisStore from 'connect-redis';
 import type { RequestHandler } from 'express';
 import type { Redis } from 'src/modules/redis/redis.providers';

src/modules/ssr-front-end/settings/settings.controller.ts

@@ -33,15 +33,15 @@ import { SettingsService } from './settings.service';
 @Controller('/account')
 export class SettingsController {
   constructor(
-    private readonly _service: SettingsService,
+    private readonly settingsService: SettingsService,
-    private readonly _form: FormUtilityService,
+    private readonly formService: FormUtilityService,
-    private readonly _upload: UploadService,
+    private readonly uploadService: UploadService,
-    private readonly _token: TokenService,
+    private readonly tokenService: TokenService,
-    private readonly _user: UserService,
+    private readonly userService: UserService,
-    private readonly _totp: UserTOTPService,
+    private readonly totpService: UserTOTPService,
-    private readonly _client: OAuth2ClientService,
+    private readonly clientService: OAuth2ClientService,
-    private readonly _oaToken: OAuth2TokenService,
+    private readonly oaTokenService: OAuth2TokenService,
-    private readonly _audit: AuditService,
+    private readonly auditService: AuditService,
   ) {}
 
   @Get()
@@ -53,7 +53,7 @@ export class SettingsController {
   @Get('general')
   @Render('settings/general')
   public general(@Req() req: Request) {
-    return this._form.populateTemplate(req, { user: req.user });
+    return this.formService.populateTemplate(req, { user: req.user });
   }
 
   @Post('general')
@@ -63,7 +63,7 @@ export class SettingsController {
     @Body() body: { display_name?: string },
   ) {
     try {
-      const { display_name } = this._form.trimmed(body, ['display_name']);
+      const { display_name } = this.formService.trimmed(body, ['display_name']);
       if (!display_name) {
         throw new Error('Display name is required.');
       }
@@ -76,7 +76,7 @@ export class SettingsController {
 
       req.user.display_name = display_name;
 
-      await this._user.updateUser(req.user);
+      await this.userService.updateUser(req.user);
       req.flash('message', {
         error: false,
         text: 'Display name has been changed!',
@@ -99,7 +99,7 @@ export class SettingsController {
     @UploadedFile() file: Express.Multer.File,
   ) {
     try {
-      if (!this._token.verifyCSRF(req)) {
+      if (!this.tokenService.verifyCSRF(req)) {
         throw new BadRequestException('Invalid session. Please try again.');
       }
 
@@ -107,15 +107,18 @@ export class SettingsController {
         throw new BadRequestException('Avatar upload failed');
       }
 
-      const matches = await this._upload.checkImageAspect(file);
+      const matches = await this.uploadService.checkImageAspect(file);
       if (!matches) {
         throw new BadRequestException(
           'Avatar should be with a 1:1 aspect ratio.',
         );
       }
 
-      const upload = await this._upload.registerUploadedFile(file, req.user);
+      const upload = await this.uploadService.registerUploadedFile(
-      await this._user.updateAvatar(req.user, upload);
+        file,
+        req.user,
+      );
+      await this.userService.updateAvatar(req.user, upload);
 
       return {
         file: upload.file,
@@ -128,7 +131,7 @@ export class SettingsController {
 
   @Post('avatar/delete')
   public async deleteUserAvatar(@Req() req: Request, @Res() res: Response) {
-    this._user.deleteAvatar(req.user);
+    this.userService.deleteAvatar(req.user);
     req.flash('message', {
       error: false,
       text: 'Avatar removed successfully.',
@@ -139,8 +142,8 @@ export class SettingsController {
   @Get('oauth2')
   @Render('settings/oauth2')
   public async authorizations(@Req() req: Request) {
-    const authorizations = await this._client.getAuthorizations(req.user);
+    const authorizations = await this.clientService.getAuthorizations(req.user);
-    return this._form.populateTemplate(req, { authorizations });
+    return this.formService.populateTemplate(req, { authorizations });
   }
 
   @Post('oauth2/revoke/:id')
@@ -149,7 +152,7 @@ export class SettingsController {
     @Res() res: Response,
     @Param('id') id: number,
   ) {
-    const getAuth = await this._client.getAuthorization(req.user, id);
+    const getAuth = await this.clientService.getAuthorization(req.user, id);
     const jsreq =
       req.header('content-type').startsWith('application/json') ||
       req.header('accept').startsWith('application/json');
@@ -169,8 +172,8 @@ export class SettingsController {
       return;
     }
 
-    await this._oaToken.wipeClientTokens(getAuth.client, req.user);
+    await this.oaTokenService.wipeClientTokens(getAuth.client, req.user);
-    await this._client.revokeAuthorization(getAuth);
+    await this.clientService.revokeAuthorization(getAuth);
 
     if (jsreq) {
       return res.json({ success: true });
@@ -186,8 +189,8 @@ export class SettingsController {
     const emailHint = `${mailSplit[0].substring(0, 1)}${asterisks}@${
       mailSplit[1]
     }`;
-    const twofactor = await this._totp.userHasTOTP(req.user);
+    const twofactor = await this.totpService.userHasTOTP(req.user);
-    return this._form.populateTemplate(req, {
+    return this.formService.populateTemplate(req, {
       user: req.user,
       emailHint,
       twofactor,
@@ -211,11 +214,13 @@ export class SettingsController {
         throw new Error('Please fill out all of the fields.');
       }
 
-      if (!(await this._user.comparePasswords(req.user.password, password))) {
+      if (
+        !(await this.tokenService.comparePasswords(req.user.password, password))
+      ) {
         throw new Error('Current password is invalid.');
       }
 
-      if (!new_password.match(this._form.passwordRegex)) {
+      if (!new_password.match(this.formService.passwordRegex)) {
         throw new Error(
           'Password must be at least 8 characters long, contain a capital and lowercase letter and a number',
         );
@@ -233,10 +238,10 @@ export class SettingsController {
       return;
     }
 
-    const newPassword = await this._user.hashPassword(new_password);
+    const newPassword = await this.tokenService.hashPassword(new_password);
     req.user.password = newPassword;
-    await this._user.updateUser(req.user);
+    await this.userService.updateUser(req.user);
-    await this._audit.auditRequest(
+    await this.auditService.auditRequest(
       req,
       AuditAction.PASSWORD_CHANGE,
       'settings',
@@ -270,13 +275,13 @@ export class SettingsController {
         throw new Error('The current email address is invalid.');
       }
 
-      if (!email.match(this._form.emailRegex)) {
+      if (!email.match(this.formService.emailRegex)) {
         throw new Error('The new email address is invalid.');
       }
 
       if (
         !current_password ||
-        !(await this._user.comparePasswords(
+        !(await this.tokenService.comparePasswords(
           req.user.password,
           current_password,
         ))
@@ -284,7 +289,7 @@ export class SettingsController {
         throw new Error('Current password is invalid.');
       }
 
-      const existing = await this._user.getByEmail(email);
+      const existing = await this.userService.getByEmail(email);
       if (existing) {
         throw new Error(
           'There is already an existing user with this email address.',
@@ -300,8 +305,12 @@ export class SettingsController {
     }
 
     req.user.email = email;
-    await this._user.updateUser(req.user);
+    await this.userService.updateUser(req.user);
-    await this._audit.auditRequest(req, AuditAction.EMAIL_CHANGE, 'settings');
+    await this.auditService.auditRequest(
+      req,
+      AuditAction.EMAIL_CHANGE,
+      'settings',
+    );
 
     req.flash('message', {
       error: false,
@@ -316,7 +325,7 @@ export class SettingsController {
     @Res() res: Response,
     @Query('csrf') csrf: string,
   ) {
-    if (!this._token.verifyCSRF(req, csrf)) {
+    if (!this.tokenService.verifyCSRF(req, csrf)) {
       throw new BadRequestException('Invalid csrf token');
     }
 
@@ -326,9 +335,12 @@ export class SettingsController {
   @Get('logins')
   @Render('login-list')
   public async userLogins(@Req() req: Request) {
-    const logins = await this._audit.getUserLogins(req.user, req.session.id);
+    const logins = await this.auditService.getUserLogins(
-    const creation = await this._audit.getUserAccountCreation(req.user);
+      req.user,
-    return this._form.populateTemplate(req, {
+      req.session.id,
+    );
+    const creation = await this.auditService.getUserAccountCreation(req.user);
+    return this.formService.populateTemplate(req, {
       logins,
       creation,
     });

src/modules/ssr-front-end/two-factor/two-factor.controller.ts

@@ -34,9 +34,8 @@ export class TwoFactorController {
     if (!twoFA) {
       const challengeString = req.query.challenge as string;
       if (challengeString) {
-        const challenge = await this.token.decryptChallenge<ChallengeType>(
+        const challenge =
-          challengeString,
+          await this.token.decryptChallenge<ChallengeType>(challengeString);
-        );
         if (
           challenge.type === 'totp' &&
           challenge.user === req.user.uuid &&
@@ -87,9 +86,8 @@ export class TwoFactorController {
         throw new Error('Invalid request');
       }
 
-      const challenge = await this.token.decryptChallenge<ChallengeType>(
+      const challenge =
-        challengeString,
+        await this.token.decryptChallenge<ChallengeType>(challengeString);
-      );
       secret = challenge.secret;
 
       if (
@@ -151,7 +149,7 @@ export class TwoFactorController {
       }
 
       if (
-        !(await this.user.comparePasswords(req.user.password, body.password))
+        !(await this.token.comparePasswords(req.user.password, body.password))
       ) {
         throw new Error('The entered password is invalid.');
       }

src/modules/utility/services/token.service.ts

@@ -1,8 +1,9 @@
 import { Injectable } from '@nestjs/common';
 import * as crypto from 'crypto';
+import * as bcrypt from 'bcrypt';
 import { ConfigurationService } from 'src/modules/config/config.service';
 import { v4 } from 'uuid';
-import * as CSRF from 'csrf';
+import CSRF from 'csrf';
 import { Request } from 'express';
 
 const IV_LENGTH = 16;
@@ -37,6 +38,18 @@ export class TokenService {
     return v4();
   }
 
+  public async comparePasswords(
+    hash: string,
+    password: string,
+  ): Promise<boolean> {
+    return bcrypt.compare(password, hash);
+  }
+
+  public async hashPassword(password: string): Promise<string> {
+    const salt = await bcrypt.genSalt(10);
+    return bcrypt.hash(password, salt);
+  }
+
   // https://stackoverflow.com/q/52212430
   /**
    * Symmetric encryption function

src/modules/well-known/well-known.controller.ts

@@ -15,25 +15,10 @@ export class WellKnownController {
   @Get('security.txt')
   securityTXT(@Res({ passthrough: true }) res: Response) {
     res.set('content-type', 'text/plain');
-    const date = new Date();
-    date.setMonth(date.getMonth() + 6);
-    date.setDate(1);
-    date.setHours(0);
-    date.setMinutes(0);
-    date.setSeconds(0);
-    date.setMilliseconds(0);
     return `# If you would like to report a security issue
 # you may report it to:
 Contact: mailto:evert@lunasqu.ee
-
+    `;
-# GnuPG public key
-Encryption: https://lunasqu.ee/public/keys/pgp/Evert%20Prants.pub
-
-# English and Estonian
-Preferred-Languages: en, et
-
-Expires: ${date.toISOString()}
-`;
   }
 
   @Get('dnt')

tsconfig.json

@@ -1,6 +1,7 @@
 {
   "compilerOptions": {
-    "module": "commonjs",
+    "module": "Node16",
+    "moduleResolution": "Node16",
     "declaration": true,
     "removeComments": true,
     "emitDecoratorMetadata": true,

views/login/login.pug

@@ -20,27 +20,11 @@ block body
         div.form-container
           input#csrf(type="hidden", name="_csrf", value=csrf)
           label.form-label(for="username") Username
-          input.form-control#username(
+          input.form-control#username(type="text", name="username", placeholder="Username", autofocus, value=form.username)
-            type="text",
-            name="username",
-            autocomplete="username",
-            placeholder="Username",
-            autofocus,
-            value=form.username
-          )
           label.form-label(for="password") Password
-          input.form-control#password(
+          input.form-control#password(type="password", name="password", placeholder="Password")
-            type="password",
-            name="password",
-            autocomplete="current-password",
-            placeholder="Password"
-          )
           div.form-checkbox
-            input.form-control#remember(
+            input.form-control#remember(type="checkbox", name="remember", checked=form.remember)
-              type="checkbox",
-              name="remember",
-              checked=form.remember
-            )
             label(for="remember") Remember me
           button.btn.btn-primary(type="submit") Log in
           div.btn-group.align-self-end

views/login/password.pug

@@ -21,35 +21,18 @@ block body
           div.form-container
             input#csrf(type="hidden", name="_csrf", value=csrf)
             label.form-label(for="password") New password
-            input.form-control#password(
+            input.form-control#password(type="password", name="password", autofocus, placeholder="Password")
-              type="password",
-              name="password",
-              autocomplete="new-password",
-              autofocus,
-              placeholder="Password"
-            )
             small.form-hint Must be at least 8 characters long, contain a capital and lowercase letter and a number.
-
             label.form-label(for="password_repeat") Repeat new password
-            input.form-control#password_repeat(
+            input.form-control#password_repeat(type="password", name="password_repeat", placeholder="Password")
-              type="password",
-              name="password_repeat",
-              autocomplete="new-password",
-              placeholder="Password"
-            )
             button.btn.btn-primary(type="submit") Set password
       else
         h1 Reset password
-        p If you have forgotten your password, please enter your account email address and we will send you a link to recover it.
+        p If you have forgotten your password, please enter your accounts email address and we will send you a link to recover it.
         form(method="post")
           div.form-container
             input#csrf(type="hidden", name="_csrf", value=csrf)
             label.form-label(for="email") Email address
-            input.form-control#email(
+            input.form-control#email(type="email", name="email", autofocus, placeholder="Email addres")
-              type="email",
-              name="email",
-              autofocus,
-              placeholder="Email address"
-            )
             button.btn.btn-primary(type="submit") Send recovery email
             a.btn.btn-link.align-self-end(type="button" href="/login") Log in instead

views/password.pug

@@ -19,19 +19,6 @@ block body
       form(method="post")
         div.form-container
           input#csrf(type="hidden", name="_csrf", value=csrf)
-          input#username(
-            type="text",
-            name="username",
-            value=user.username,
-            autocomplete="username",
-            style="display: none"
-          )
-
           label.form-label(for="password") Password
-          input.form-control#password(
+          input.form-control#password(type="password", name="password", autofocus)
-            type="password",
-            name="password",
-            autofocus,
-            autocomplete="current-password"
-          )
           button.btn.btn-primary(type="submit") Submit

views/register.pug

@@ -22,51 +22,23 @@ block body
             input#csrf(type="hidden", name="_csrf", value=csrf)
 
             label.form-label(for="username") Username
-            input.form-control#username(
+            input.form-control#username(type="text", name="username", placeholder="Username", autofocus, value=form.username)
-              type="text",
-              name="username",
-              placeholder="Username",
-              autocomplete="username",
-              autofocus,
-              value=form.username
-            )
             small.form-hint Between 3 and 26 English alphanumeric characters and .-_ only.
 
             label.form-label(for="display_name") Display name
-            input.form-control#display_name(
+            input.form-control#display_name(type="text", name="display_name", placeholder="Display name", value=form.display_name)
-              type="text",
-              name="display_name",
-              placeholder="Display name",
-              value=form.display_name
-            )
             small.form-hint Maximum length is 32.
 
             label.form-label(for="email") Email address
-            input.form-control#email(
+            input.form-control#email(type="email", name="email", placeholder="Email address", value=form.email)
-              type="email",
-              name="email",
-              placeholder="Email address",
-              value=form.email
-            )
             small.form-hint You will need to verify your email address before you can log in.
 
             label.form-label(for="password") Password
-            input.form-control#password(
+            input.form-control#password(type="password", name="password", placeholder="Password", value=form.password)
-              type="password",
-              name="password",
-              autocomplete="new-password",
-              placeholder="Password",
-              value=form.password
-            )
             small.form-hint Must be at least 8 characters long, contain a capital and lowercase letter and a number.
 
             label.form-label(for="password_repeat") Confirm password
-            input.form-control#password_repeat(
+            input.form-control#password_repeat(type="password", name="password_repeat", placeholder="Confirm password")
-              type="password",
-              name="password_repeat",
-              autocomplete="new-password",
-              placeholder="Confirm password"
-            )
 
             button.btn.btn-primary(type="submit") Create a new account
             a.btn.btn-link.align-self-end(type="button" href="/login") Log in instead

views/settings/security.pug

@@ -18,23 +18,21 @@ block settings
       form(method="post", action="/account/security/password", autocomplete="off")
         div.form-container
           input#csrf(type="hidden", name="_csrf", value=csrf)
-          input#username(type="text", name="username", value=user.username, autocomplete="username", style="display: none")
           label.form-label(for="password") Current Password
-          input.form-control#password(type="password", name="password", autocomplete="current-password")
+          input.form-control#password(type="password", name="password")
           label.form-label(for="new_password") New Password
           input.form-control#new_password(type="password", name="new_password", autocomplete="new-password")
           small.form-hint At least 8 characters, a capital letter and a number required.
           label.form-label(for="password_repeat") Repeat new password
-          input.form-control#password_repeat(type="password", name="password_repeat", autocomplete="new-password")
+          input.form-control#password_repeat(type="password", name="password_repeat")
           button.btn.btn-primary(type="submit") Change
     .col
       h2 Change Email Address
       form(method="post", action="/account/security/email", autocomplete="off")
         div.form-container
           input(type="hidden", name="_csrf", value=csrf)
-          input#email_username(type="text", name="username", value=user.username, autocomplete="username", style="display: none")
           label.form-label(for="current_password") Current Password
-          input.form-control#current_password(type="password", name="current_password", autocomplete="current-password")
+          input.form-control#current_password(type="password", name="current_password")
           label.form-label(for="current_email") Current Email Address
           input.form-control#current_email(type="email", name="current_email")
           small.form-hint Hint: #{emailHint}