feat(profile): add censor-sensor to stop people from being unnecessar…

…ily profane. closes #11
After-the-End-of-All-Things · Jun 27, 2023 · 429fbf8 · 429fbf8
1 parent bfc618f
commit 429fbf8
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 6 deletions.
diff --git a/server/src/modules/auth/auth.service.ts b/server/src/modules/auth/auth.service.ts
@@ -8,7 +8,7 @@ import * as bcrypt from 'bcrypt';
 import { sample } from 'lodash';
 
 import { IFullUser, IHasAccessToken } from '@interfaces';
-import { AnalyticsService } from '@modules/content/analytics.service';
+import { ContentService } from '@modules/content/content.service';
 import { User } from '../user/user.schema';
 import { UserService } from '../user/user.service';
 
@@ -17,14 +17,19 @@ export class AuthService {
   constructor(
     private readonly userService: UserService,
     private readonly jwtService: JwtService,
-    private readonly analyticsService: AnalyticsService,
+    private readonly contentService: ContentService,
   ) {}
 
   async signUp(
     username: string,
     password: string,
     email: string,
   ): Promise<IFullUser> {
+    if (this.contentService.censor.isProfaneIsh(username))
+      throw new BadRequestException(
+        'Username is somewhat profane. Please choose again.',
+      );
+
     const usersWithUsername = await this.userService.getAllUsersWithUsername(
       username,
     );

diff --git a/server/src/modules/content/content.service.ts b/server/src/modules/content/content.service.ts
@@ -4,8 +4,17 @@ import { ICollectible, IEquipment, IItem, IJob, ILocation } from '@interfaces';
 import * as fs from 'fs-extra';
 import { Logger } from 'nestjs-pino';
 
+import { CensorSensor } from 'censor-sensor';
+
+const censor = new CensorSensor();
+censor.disableTier(4);
+
 @Injectable()
 export class ContentService {
+  public get censor(): CensorSensor {
+    return censor;
+  }
+
   public content = { locations: {}, jobs: {}, collectibles: {}, equipment: {} };
 
   private get locations(): Record<string, ILocation> {

diff --git a/server/src/modules/player/player.service.ts b/server/src/modules/player/player.service.ts
@@ -90,9 +90,11 @@ export class PlayerService {
     const player = await this.getPlayerForUser(userId);
     if (!player) throw new ForbiddenException('Player not found');
 
-    shortBio = shortBio.substring(0, 30).trim();
+    shortBio = this.contentService.censor.cleanProfanityIsh(
+      shortBio.substring(0, 30).trim(),
+    );
 
-    if(player.profile.shortBio === shortBio) return {};
+    if (player.profile.shortBio === shortBio) return {};
 
     const playerPatches = await getPatchesAfterPropChanges<Player>(
       player,
@@ -120,9 +122,11 @@ export class PlayerService {
     const player = await this.getPlayerForUser(userId);
     if (!player) throw new ForbiddenException('Player not found');
 
-    longBio = longBio.substring(0, 500).trim();
+    longBio = this.contentService.censor.cleanProfanityIsh(
+      longBio.substring(0, 500).trim(),
+    );
 
-    if(player.profile.longBio === longBio) return {};
+    if (player.profile.longBio === longBio) return {};
 
     const playerPatches = await getPatchesAfterPropChanges<Player>(
       player,