fix(my-pages): hotfix for subpoenas

charts/islandis/values.prod.yaml

@@ -356,7 +356,7 @@ api:
     XROAD_ICELAND_UNIVERSITY_OF_THE_ARTS_PATH: 'IS/EDU/4210984099/LHI-Protected/brautskraning-v1'
     XROAD_INNA_PATH: 'IS/GOV/6601241280/MMS-Protected/inna-v1'
     XROAD_INTELLECTUAL_PROPERTIES_PATH: 'IS/GOV/6501912189/WebAPI-Public/HUG-webAPI/'
-    XROAD_JUDICIAL_SYSTEM_SP_PATH: 'IS-GOV/GOV/5804170510/Rettarvorslugatt-Private/judicial-system-mailbox-api'
+    XROAD_JUDICIAL_SYSTEM_SP_PATH: 'IS/GOV/5804170510/Rettarvorslugatt-Private/judicial-system-mailbox-api'
     XROAD_MMS_FRIGG_PATH: 'IS/GOV/10066/MMS-Protected/frigg-form-service'
     XROAD_MMS_GRADE_SERVICE_ID: 'IS/GOV/6601241280/MMS-Protected/grade-api-v1'
     XROAD_MMS_LICENSE_SERVICE_ID: 'IS/GOV/6601241280/MMS-Protected/license-api-v1'

infra/src/dsl/xroad.ts

@@ -847,7 +847,7 @@ export const JudicialSystemServicePortal = new XroadConf({
       dev: 'IS-DEV/GOV/10014/Rettarvorslugatt-Private/judicial-system-mailbox-api',
       staging:
         'IS-TEST/GOV/10014/Rettarvorslugatt-Private/judicial-system-mailbox-api',
-      prod: 'IS-GOV/GOV/5804170510/Rettarvorslugatt-Private/judicial-system-mailbox-api',
+      prod: 'IS/GOV/5804170510/Rettarvorslugatt-Private/judicial-system-mailbox-api',
     },
   },
 })

libs/api/domains/documents/src/lib/document.types.ts

@@ -1,6 +1,7 @@
 import { FileType } from './models/v2/documentContent.model'
 
 export const HEALTH_CATEGORY_ID = '3'
+export const LAW_AND_ORDER_CATEGORY_ID = '12'
 
 export class DocumentTypeFilter {
   senderName!: string

libs/api/domains/documents/src/lib/documentV2.resolver.ts

@@ -36,6 +36,7 @@ import { DocumentV2MarkAllMailAsRead } from './models/v2/markAllMailAsRead.model
 import type { Locale } from '@island.is/shared/types'
 import { DocumentConfirmActionsInput } from './models/v2/confirmActions.input'
 import { DocumentConfirmActions } from './models/v2/confirmActions.model'
+import { isDefined } from '@island.is/shared/utils'
 
 const LOG_CATEGORY = 'documents-resolver'
 
@@ -95,9 +96,8 @@ export class DocumentResolverV2 {
     @CurrentUser() user: User,
   ): Promise<PaginatedDocuments> {
     const ffEnabled = await this.getFeatureFlag()
-    if (ffEnabled)
-      return this.documentServiceV2.listDocumentsV3(user.nationalId, input)
-    return this.documentServiceV2.listDocuments(user.nationalId, input)
+    if (ffEnabled) return this.documentServiceV2.listDocumentsV3(user, input)
+    return this.documentServiceV2.listDocuments(user, input)
   }
 
   @Scopes(DocumentsScope.main)
@@ -121,8 +121,13 @@ export class DocumentResolverV2 {
   }
 
   @ResolveField('categories', () => [Category])
-  documentCategories(@CurrentUser() user: User): Promise<Array<Category>> {
-    return this.documentServiceV2.getCategories(user.nationalId)
+  async documentCategories(
+    @CurrentUser() user: User,
+  ): Promise<Array<Category>> {
+    const categories = await this.documentServiceV2.getCategories(user)
+    if (!isDefined(categories)) {
+      return []
+    } else return categories
   }
 
   @ResolveField('senders', () => [Sender])

libs/api/domains/documents/src/lib/documentV2.service.ts

@@ -18,10 +18,14 @@ import { DocumentsInput } from './models/v2/documents.input'
 import { PaperMailPreferences } from './models/v2/paperMailPreferences.model'
 import { Sender } from './models/v2/sender.model'
 import { FileType } from './models/v2/documentContent.model'
-import { HEALTH_CATEGORY_ID } from './document.types'
+import { HEALTH_CATEGORY_ID, LAW_AND_ORDER_CATEGORY_ID } from './document.types'
 import { Type } from './models/v2/type.model'
 import { DownloadServiceConfig } from '@island.is/nest/config'
 import { DocumentV2MarkAllMailAsRead } from './models/v2/markAllMailAsRead.model'
+import type { User } from '@island.is/auth-nest-tools'
+import { AuthDelegationType } from '@island.is/shared/types'
+import { getBirthday } from './helpers/getBirthday'
+import differceInYears from 'date-fns/differenceInYears'
 import type { Locale } from '@island.is/shared/types'
 
 const LOG_CATEGORY = 'documents-api-v2'
@@ -128,6 +132,12 @@ export class DocumentServiceV2 {
     const actions = document.actions?.filter(
       (action) => action.type !== 'alert' && action.type !== 'confirmation',
     )
+    if (document.urgent)
+      this.logger.info('Urgent document fetched', {
+        documentId: documentId,
+        includeDocument,
+      })
+
     return {
       ...document,
       publicationDate: document.date,
@@ -152,30 +162,29 @@ export class DocumentServiceV2 {
   }
 
   async listDocuments(
-    nationalId: string,
+    user: User,
     input: DocumentsInput,
   ): Promise<PaginatedDocuments> {
     //If a delegated user is viewing the mailbox, do not return any health related data
     //Category is now "1,2,3,...,n"
     const { categoryIds, ...restOfInput } = input
     let mutableCategoryIds = categoryIds ?? []
-
-    if (input.isLegalGuardian) {
-      if (!mutableCategoryIds.length) {
-        mutableCategoryIds = (await this.getCategories(nationalId, true)).map(
-          (c) => c.id,
-        )
-      } else {
-        mutableCategoryIds = mutableCategoryIds.filter(
-          (c) => c === HEALTH_CATEGORY_ID,
-        )
+    if (!mutableCategoryIds.length) {
+      const filteredCategories = await this.getCategories(user)
+      if (isDefined(filteredCategories)) {
+        mutableCategoryIds = filteredCategories.map((c) => c.id)
       }
     }
+    // If categoryIds are provided, filter out correct data
+    else {
+      const hiddenCategoryIds = this.getHiddenCategoriesIDs(user)
+      mutableCategoryIds.filter((c) => !hiddenCategoryIds.includes(c))
+    }
 
     const documents = await this.documentService.getDocumentList({
       ...restOfInput,
       categoryId: mutableCategoryIds.join(),
-      nationalId,
+      nationalId: user.nationalId,
     })
 
     if (typeof documents?.totalCount !== 'number') {
@@ -215,30 +224,32 @@ export class DocumentServiceV2 {
   }
 
   async listDocumentsV3(
-    nationalId: string,
+    user: User,
     input: DocumentsInput,
   ): Promise<PaginatedDocuments> {
     //If a delegated user is viewing the mailbox, do not return any health related data
     //Category is now "1,2,3,...,n"
     const { categoryIds, ...restOfInput } = input
+    // If no categoryIds are provided, get all categories
     let mutableCategoryIds = categoryIds ?? []
 
-    if (input.isLegalGuardian) {
-      if (!mutableCategoryIds.length) {
-        mutableCategoryIds = (await this.getCategories(nationalId, true)).map(
-          (c) => c.id,
-        )
-      } else {
-        mutableCategoryIds = mutableCategoryIds.filter(
-          (c) => c === HEALTH_CATEGORY_ID,
-        )
+    if (!mutableCategoryIds.length) {
+      const filteredCategories = await this.getCategories(user)
+      if (isDefined(filteredCategories)) {
+        mutableCategoryIds = filteredCategories.map((c) => c.id)
       }
     }
+    // If categoryIds are provided, filter out correct data
+    else {
+      const hiddenCategoryIds = this.getHiddenCategoriesIDs(user)
+      mutableCategoryIds.filter((c) => !hiddenCategoryIds.includes(c))
+    }
 
     const documents = await this.documentService.getDocumentList({
       ...restOfInput,
       categoryId: mutableCategoryIds.join(),
-      nationalId,
+      hiddenCategoryIds: this.getHiddenCategoriesIDs(user).join(),
+      nationalId: user.nationalId,
     })
 
     if (typeof documents?.totalCount !== 'number') {
@@ -268,7 +279,7 @@ export class DocumentServiceV2 {
         .filter(isDefined) ?? []
 
     return {
-      data: documentData,
+      data: documentData, // if empty after category filter then return null
       totalCount: documents?.totalCount ?? 0,
       unreadCount: documents?.unreadCount,
       pageInfo: {
@@ -277,27 +288,32 @@ export class DocumentServiceV2 {
     }
   }
 
-  async getCategories(
-    nationalId: string,
-    filterHealth = false,
-  ): Promise<Array<Category>> {
+  async getCategories(user: User): Promise<Array<Category> | null> {
     const categories = await this.documentService.getCustomersCategories(
-      nationalId,
+      user.nationalId,
     )
 
-    return (
+    const filterOutIds = this.getHiddenCategoriesIDs(user) ?? []
+
+    const filteredCategories =
       categories.categories
         ?.map((c) => {
-          if (!c.id || (filterHealth && c.id === HEALTH_CATEGORY_ID)) {
+          if (
+            !c.id ||
+            (filterOutIds.length > 0 && filterOutIds.includes(c.id))
+          ) {
             return null
-          }
-          return {
-            id: c.id,
-            name: c.name,
+          } else {
+            return {
+              id: c.id,
+              name: c.name,
+            }
           }
         })
         .filter(isDefined) ?? []
-    )
+    if (filteredCategories.length === 0) return null
+
+    return filteredCategories
   }
 
   async getTypes(nationalId: string): Promise<Array<Type>> {
@@ -484,7 +500,7 @@ export class DocumentServiceV2 {
 
     // we return the document even if the actions are faulty, logged for tracability
     if (hasEmpty) {
-      this.logger.warn('No title or data in actions array', {
+      this.logger.info('No title or data in actions array, return undefined', {
         category: LOG_CATEGORY,
         id,
       })
@@ -511,6 +527,32 @@ export class DocumentServiceV2 {
       }
     })
 
+    // Log the actions to ensure that they are mapped correctly
+    this.logger.info('Actions mapped successfully', { actions: mapped })
     return mapped
   }
+
+  private getHiddenCategoriesIDs = (user: User) => {
+    const isDelegated = isDefined(user.delegationType)
+    if (!isDelegated) return []
+
+    const isLegalGuardian = user.delegationType?.includes(
+      AuthDelegationType.LegalGuardian,
+    )
+    const birthdate = getBirthday(user.nationalId)
+    const childAgeIs16OrOlder = birthdate
+      ? differceInYears(new Date(), birthdate) > 15
+      : false
+
+    // Hide health data if user is a legal guardian and child is 16 or older
+    const hideHealthData = isLegalGuardian && childAgeIs16OrOlder
+    // Hide law and order data if user is delegated
+    // commented out until we have correct category for law and order files
+    const hideLawAndOrderData = isDelegated
+
+    return [
+      ...(hideHealthData ? [HEALTH_CATEGORY_ID] : []),
+      ...(hideLawAndOrderData ? [LAW_AND_ORDER_CATEGORY_ID] : []),
+    ]
+  }
 }

libs/api/domains/documents/src/lib/models/v2/documents.input.ts

@@ -101,9 +101,4 @@ export class DocumentsInput {
   @IsOptional()
   @IsInt()
   readonly pageSize?: number
-
-  @Field({ nullable: true })
-  @IsOptional()
-  @IsBoolean()
-  isLegalGuardian?: boolean
 }

libs/api/domains/law-and-order/src/lib/law-and-order.resolver.ts

@@ -95,7 +95,12 @@ export class LawAndOrderResolver {
     @Args('locale', { type: () => String, nullable: true })
     locale: Locale = 'is',
   ) {
-    return this.lawAndOrderService.getLawyers(user, locale)
+    return this.auditAndHandle(
+      'getLawyers',
+      'lawyers',
+      this.lawAndOrderService.getLawyers(user, locale),
+      user,
+    )
   }
 
   @Mutation(() => DefenseChoice, {

libs/auth/react/src/lib/auth/Auth.state.ts

@@ -1,5 +1,4 @@
 import { User } from '@island.is/shared/types'
-import { getBirthday } from '../utils/getBirthday'
 
 export type AuthState =
   | 'logged-out'
@@ -37,20 +36,6 @@ export const initialState: AuthReducerState = {
   isAuthenticated: false,
 }
 
-// Add dateOfBirth Date object to user profile
-const formatUser = (payload: User): User | null => {
-  const dateOfBirth = getBirthday(payload?.profile?.nationalId)
-
-  return {
-    ...payload,
-    scopes: payload.scopes || [],
-    profile: {
-      ...payload.profile,
-      dateOfBirth,
-    },
-  }
-}
-
 export const reducer = (
   state: AuthReducerState,
   action: Action,
@@ -64,15 +49,16 @@ export const reducer = (
     case ActionType.SIGNIN_SUCCESS:
       return {
         ...state,
-        userInfo: formatUser(action.payload),
+        userInfo: action.payload,
+
         authState: 'logged-in',
         isAuthenticated: true,
       }
     case ActionType.USER_LOADED:
       return state.isAuthenticated
         ? {
             ...state,
-            userInfo: formatUser(action.payload),
+            userInfo: action.payload,
           }
         : state
     case ActionType.SIGNIN_FAILURE:

libs/clients/documents-v2/src/lib/documentsClientV2.service.ts

@@ -31,6 +31,7 @@ export class DocumentsClientV2Service {
      * @param input List input object. Example: { dateFrom: undefined, nationalId: '123' }
      * @returns List object sanitized of unnecessary values. Example: { nationalId: '123' }
      */
+
     const sanitizeObject = function <T extends { [key: string]: any }>(
       obj: T,
     ): T {
@@ -43,6 +44,15 @@ export class DocumentsClientV2Service {
       return sanitizedObj
     }
 
+    // If hiddenCats is not empty and categories are empty, the API will return no documents
+    if (input.hiddenCategoryIds !== '' && input.categoryId === '') {
+      return {
+        totalCount: 0,
+        unreadCount: 0,
+        documents: [],
+      }
+    }
+
     const inputObject = sanitizeObject({
       ...input,
       kennitala: input.nationalId,