fix: standardize regex validation pattern for username change

Author: Bentroen

shared/validation/user/constants.ts

@@ -3,4 +3,5 @@ import { deepFreeze } from '../common/deepFreeze';
 export const UserConst = deepFreeze({
   USERNAME_MIN_LENGTH: 3,
   USERNAME_MAX_LENGTH: 32,
+  ALLOWED_REGEXP: /^[a-zA-Z0-9-_.]*$/,
 });

shared/validation/user/dto/UpdateUsername.dto.ts

@@ -1,11 +1,12 @@
 import { ApiProperty } from '@nestjs/swagger';
-import { IsString, IsNotEmpty, MinLength, MaxLength } from 'class-validator';
+import { IsString, MinLength, MaxLength, Matches } from 'class-validator';
 import { UserConst } from '../constants';
 
 export class UpdateUsernameDto {
   @IsString()
   @MaxLength(UserConst.USERNAME_MAX_LENGTH)
   @MinLength(UserConst.USERNAME_MIN_LENGTH)
+  @Matches(UserConst.ALLOWED_REGEXP)
   @ApiProperty({
     description: 'Username of the user',
     example: 'tomast1137',

web/src/modules/shared/components/layout/UserMenu.tsx

@@ -142,7 +142,7 @@ export const UserMenu = ({ userData }: { userData: LoggedUserData }) => {
                         {...register('username', {
                           required: 'Username is required',
                           pattern: {
-                            value: /^[a-zA-Z0-9-_.]*$/,
+                            value: UserConst.ALLOWED_REGEXP,
                             message:
                               'Your username may only contain these characters: A-Z a-z 0-9 - _ .',
                           },