chore(user-profile): Remove migration worker and make user profile strict

apps/services/user-profile/infra/service-portal-api.ts

@@ -9,13 +9,8 @@ import {
   NationalRegistryB2C,
 } from '../../../../infra/src/dsl/xroad'
 
-// We basically don't want it to run in a cron job
-// but manually, so set it to run once a year on dec 31st
-const schedule = '0 0 31 12 *'
-
 const namespace = 'service-portal'
 const serviceId = `${namespace}-api`
-const workerId = `${serviceId}-worker`
 const imageId = 'services-user-profile'
 
 const envVariables: EnvironmentVariables = {
@@ -36,11 +31,6 @@ const envVariables: EnvironmentVariables = {
     staging: 'false',
     prod: 'false',
   },
-  USER_PROFILE_WORKER_PAGE_SIZE: {
-    dev: '3000',
-    staging: '3000',
-    prod: '3000',
-  },
   AUTH_DELEGATION_API_URL: {
     dev: 'http://web-services-auth-delegation-api.identity-server-delegation.svc.cluster.local',
     staging:
@@ -68,32 +58,6 @@ const secrets: Secrets = {
     '/k8s/api/NATIONAL_REGISTRY_B2C_CLIENT_SECRET',
 }
 
-export const workerSetup = (): ServiceBuilder<typeof workerId> =>
-  service(workerId)
-    .namespace(namespace)
-    .image(imageId)
-    .env(envVariables)
-    .secrets(secrets)
-    .files({ filename: 'islyklar.p12', env: 'ISLYKILL_CERT' })
-    .command('node')
-    .args('main.js', '--job=worker')
-    .resources({
-      limits: { cpu: '800m', memory: '1024Mi' },
-      requests: { cpu: '400m', memory: '512Mi' },
-    })
-    .db()
-    .extraAttributes({
-      dev: {
-        schedule,
-      },
-      staging: {
-        schedule,
-      },
-      prod: {
-        schedule,
-      },
-    })
-
 export const serviceSetup = (): ServiceBuilder<typeof serviceId> =>
   service(serviceId)
     .namespace(namespace)

apps/services/user-profile/migrations/20240513221742-remove-migration-table.js

@@ -0,0 +1,15 @@
+'use strict'
+
+module.exports = {
+  async up(queryInterface) {
+    return queryInterface.dropTable('user_profile_advania')
+  },
+
+  async down(queryInterface, Sequelize) {
+    /**
+     * This migration is cleaning up the migration table used when we migrated from the old user profile database.
+     *
+     * There is no down action as we cannot reinstate the data that was dropped along with the table.
+     */
+  },
+}

apps/services/user-profile/project.json

@@ -52,13 +52,6 @@
         "buildTarget": "services-user-profile:build"
       }
     },
-    "worker": {
-      "executor": "@nx/js:node",
-      "options": {
-        "buildTarget": "services-user-profile:build",
-        "args": ["--job", "worker"]
-      }
-    },
     "lint": {
       "executor": "@nx/linter:eslint",
       "options": {
@@ -98,7 +91,7 @@
     "migrate/generate": {
       "executor": "nx:run-commands",
       "options": {
-        "command": "../../../node_modules/.bin/sequelize-cli migration:generate --name $(whoami)",
+        "command": "../../../node_modules/.bin/sequelize-cli migration:generate",
         "cwd": "apps/services/user-profile/"
       }
     },

apps/services/user-profile/src/app/user-profile/dto/confirmEmailDto.ts

@@ -1,11 +1,11 @@
 import { IsNotEmpty, IsString } from 'class-validator'
-import { ApiProperty } from '@nestjs/swagger'
+import { ApiProperty, ApiPropertyOptional } from '@nestjs/swagger'
 
 export class ConfirmEmailDto {
   @IsNotEmpty()
   @IsString()
-  @ApiProperty()
-  readonly hash!: string
+  @ApiPropertyOptional()
+  readonly hash?: string
 
   @IsNotEmpty()
   @IsString()

apps/services/user-profile/src/app/user-profile/dto/confirmSmsDto.ts

@@ -1,11 +1,11 @@
 import { IsNotEmpty, IsString } from 'class-validator'
-import { ApiProperty } from '@nestjs/swagger'
+import { ApiProperty, ApiPropertyOptional } from '@nestjs/swagger'
 
 export class ConfirmSmsDto {
   @IsNotEmpty()
   @IsString()
-  @ApiProperty()
-  readonly code!: string
+  @ApiPropertyOptional()
+  readonly code?: string
 
   @IsNotEmpty()
   @IsString()

apps/services/user-profile/src/app/user-profile/emailVerification.model.ts

@@ -7,6 +7,11 @@ import {
   UpdatedAt,
 } from 'sequelize-typescript'
 import { ApiProperty } from '@nestjs/swagger'
+import type {
+  CreationOptional,
+  InferAttributes,
+  InferCreationAttributes,
+} from 'sequelize'
 
 @Table({
   tableName: 'email_verification',
@@ -17,23 +22,26 @@ import { ApiProperty } from '@nestjs/swagger'
     },
   ],
 })
-export class EmailVerification extends Model<EmailVerification> {
+export class EmailVerification extends Model<
+  InferAttributes<EmailVerification>,
+  InferCreationAttributes<EmailVerification>
+> {
   @Column({
     type: DataType.UUID,
     primaryKey: true,
     allowNull: false,
     defaultValue: DataType.UUIDV4,
   })
   @ApiProperty()
-  id!: string
+  id!: CreationOptional<string>
 
   @CreatedAt
   @ApiProperty()
-  created!: Date
+  created!: CreationOptional<Date>
 
   @UpdatedAt
   @ApiProperty()
-  modified!: Date
+  modified!: CreationOptional<Date>
 
   @Column({
     type: DataType.STRING,
@@ -47,7 +55,7 @@ export class EmailVerification extends Model<EmailVerification> {
     type: DataType.BOOLEAN,
   })
   @ApiProperty()
-  confirmed!: boolean
+  confirmed!: CreationOptional<boolean>
 
   @Column({
     type: DataType.STRING,

apps/services/user-profile/src/app/user-profile/smsVerification.model.ts

@@ -7,6 +7,11 @@ import {
   UpdatedAt,
 } from 'sequelize-typescript'
 import { ApiProperty } from '@nestjs/swagger'
+import type {
+  CreationOptional,
+  InferAttributes,
+  InferCreationAttributes,
+} from 'sequelize'
 
 @Table({
   tableName: 'sms_verification',
@@ -17,23 +22,26 @@ import { ApiProperty } from '@nestjs/swagger'
     },
   ],
 })
-export class SmsVerification extends Model<SmsVerification> {
+export class SmsVerification extends Model<
+  InferAttributes<SmsVerification>,
+  InferCreationAttributes<SmsVerification>
+> {
   @Column({
     type: DataType.UUID,
     primaryKey: true,
     allowNull: false,
     defaultValue: DataType.UUIDV4,
   })
   @ApiProperty()
-  id!: string
+  id!: CreationOptional<string>
 
   @CreatedAt
   @ApiProperty()
-  created!: Date
+  created!: CreationOptional<Date>
 
   @UpdatedAt
   @ApiProperty()
-  modified!: Date
+  modified!: CreationOptional<Date>
 
   @Column({
     type: DataType.STRING,
@@ -53,7 +61,7 @@ export class SmsVerification extends Model<SmsVerification> {
     type: DataType.BOOLEAN,
   })
   @ApiProperty()
-  confirmed!: boolean
+  confirmed!: CreationOptional<boolean>
 
   @Column({
     type: DataType.STRING,

apps/services/user-profile/src/app/user-profile/userDeviceTokens.model.ts

@@ -6,6 +6,11 @@ import {
   CreatedAt,
   UpdatedAt,
 } from 'sequelize-typescript'
+import type {
+  CreationOptional,
+  InferAttributes,
+  InferCreationAttributes,
+} from 'sequelize'
 
 @Table({
   tableName: 'user_device_tokens',
@@ -16,20 +21,23 @@ import {
     },
   ],
 })
-export class UserDeviceTokens extends Model<UserDeviceTokens> {
+export class UserDeviceTokens extends Model<
+  InferAttributes<UserDeviceTokens>,
+  InferCreationAttributes<UserDeviceTokens>
+> {
   @Column({
     type: DataType.UUID,
     primaryKey: true,
     allowNull: false,
     defaultValue: DataType.UUIDV4,
   })
-  id!: string
+  id!: CreationOptional<string>
 
   @CreatedAt
-  created!: Date
+  created!: CreationOptional<Date>
 
   @UpdatedAt
-  modified!: Date
+  modified!: CreationOptional<Date>
 
   @Column({
     type: DataType.STRING,

apps/services/user-profile/src/app/user-profile/userProfile.controller.ts

@@ -21,7 +21,6 @@ import {
   HttpCode,
   Delete,
   Patch,
-  NotFoundException,
 } from '@nestjs/common'
 import { NoContentException } from '@island.is/nest/problem'
 import {
@@ -49,6 +48,7 @@ import { UserProfileService } from './userProfile.service'
 import { VerificationService } from './verification.service'
 import { DataStatus } from './types/dataStatusTypes'
 import { ActorLocale } from './dto/actorLocale'
+import { Locale } from './types/localeTypes'
 
 @UseGuards(IdsUserGuard, ScopesGuard)
 @ApiTags('User Profile')
@@ -113,7 +113,7 @@ export class UserProfileController {
 
     return {
       nationalId: userProfile.nationalId,
-      locale: userProfile.locale,
+      locale: userProfile.locale ?? Locale.ICELANDIC,
     }
   }
 
@@ -201,8 +201,7 @@ export class UserProfileController {
     try {
       return await this.findOneByNationalId(nationalId, user)
     } catch (error) {
-      const ret = await this.create({ nationalId }, user)
-      return ret
+      return this.create({ nationalId }, user)
     }
   }
 

apps/services/user-profile/src/app/user-profile/userProfile.service.ts

@@ -25,13 +25,6 @@ export class UserProfileService {
     private readonly userDeviceTokensModel: typeof UserDeviceTokens,
   ) {}
 
-  async findById(id: string): Promise<UserProfile | null> {
-    this.logger.debug(`Finding user profile by id "${id}"`)
-    return this.userProfileModel.findOne({
-      where: { id },
-    })
-  }
-
   async create(userProfileDto: CreateUserProfileDto): Promise<UserProfile> {
     return this.userProfileModel.create({ ...userProfileDto })
   }
@@ -74,7 +67,7 @@ export class UserProfileService {
         ...body,
         nationalId: user.nationalId,
       })
-    } catch (e: any) {
+    } catch (e) {
       throw new BadRequestException(e.errors)
     }
   }

apps/services/user-profile/src/app/user-profile/userToken.controller.ts

@@ -46,7 +46,7 @@ export class UserTokenController {
   async findOneByNationalId(
     @Param('nationalId')
     nationalId: string,
-  ): Promise<UserProfile> {
+  ): Promise<UserProfile | null> {
     const userProfile = await this.userProfileService.findByNationalId(
       nationalId,
     )

apps/services/user-profile/src/app/utils/format-phone-number.ts

@@ -7,7 +7,7 @@ import { parsePhoneNumber } from 'libphonenumber-js'
  *
  * @param phoneNumber
  */
-export const formatPhoneNumber = (phoneNumber: string): string => {
+export const formatPhoneNumber = (phoneNumber?: string): string => {
   if (!phoneNumber) {
     return ''
   }

apps/services/user-profile/src/app/v2/dto/actor-profile.dto.ts

@@ -32,7 +32,7 @@ export class ActorProfileDto {
   @ApiPropertyOptional()
   @IsOptional()
   @IsEmail()
-  readonly email?: string
+  readonly email?: string | null
 
   @ApiProperty()
   @IsBoolean()
@@ -45,7 +45,7 @@ export class ActorProfileDto {
   @ApiPropertyOptional()
   @IsOptional()
   @IsEnum(Locale)
-  readonly locale?: Locale
+  readonly locale?: Locale | null
 }
 
 export class PatchActorProfileDto {