From 7f3a4d73a2a5b8f0f884fb1097a9364c1981d517 Mon Sep 17 00:00:00 2001 From: Jez Barnsley Date: Wed, 29 Apr 2026 16:49:31 +0100 Subject: [PATCH] createLogger() as a singleton --- src/api/entitlements/service.js | 3 +-- src/api/entitlements/service.test.js | 2 +- src/api/forms/repositories/form-definition-repository.js | 4 +--- src/api/forms/repositories/form-metadata-repository.js | 4 +--- .../forms/repositories/form-metadata-repository.test.js | 4 ++-- src/api/forms/repositories/form-versions-repository.js | 4 +--- src/api/forms/repositories/helpers.js | 4 +--- src/api/forms/repositories/secrets-repository.js | 4 +--- src/api/forms/repositories/secrets-repository.test.js | 4 ++-- src/api/forms/service/component.js | 2 +- src/api/forms/service/conditions.js | 2 +- src/api/forms/service/definition.js | 7 ++----- src/api/forms/service/helpers/definition.js | 2 +- src/api/forms/service/helpers/definition.test.js | 8 ++++++++ src/api/forms/service/index.js | 2 +- src/api/forms/service/lists.js | 2 +- src/api/forms/service/migration-helpers.test.js | 8 ++++++++ src/api/forms/service/migration.js | 2 +- src/api/forms/service/options.js | 2 +- src/api/forms/service/page.js | 3 ++- src/api/forms/service/report-overview.js | 2 +- src/api/forms/service/report-timeline.js | 2 +- src/api/forms/service/secrets.js | 2 +- src/api/forms/service/sections.js | 2 +- src/api/forms/service/shared.js | 3 --- src/api/forms/service/versioning.js | 4 +--- src/helpers/fail-action.js | 4 +--- src/helpers/feedback-form/reinstate.test.js | 6 +++--- src/helpers/logging/logger.js | 6 +++++- src/index.js | 4 +--- src/messaging/publish-base.js | 2 +- src/messaging/publish-base.test.js | 8 +++++++- src/plugins/auth/auth.test.js | 4 ++-- src/plugins/auth/index.js | 4 +--- src/server.js | 4 +--- 35 files changed, 65 insertions(+), 65 deletions(-) diff --git a/src/api/entitlements/service.js b/src/api/entitlements/service.js index f6863168..d689ab5c 100644 --- a/src/api/entitlements/service.js +++ b/src/api/entitlements/service.js @@ -1,12 +1,11 @@ import Boom from '@hapi/boom' import { config } from '~/src/config/index.js' -import { createLogger } from '~/src/helpers/logging/logger.js' +import { logger } from '~/src/helpers/logging/logger.js' import { getJson } from '~/src/lib/fetch.js' const entitlementUrl = config.get('entitlementUrl') const entitlementsEndpoint = new URL('/', entitlementUrl) -const logger = createLogger() /** * Fetches user scopes from the entitlements API diff --git a/src/api/entitlements/service.test.js b/src/api/entitlements/service.test.js index 09ab8775..aa2aeb31 100644 --- a/src/api/entitlements/service.test.js +++ b/src/api/entitlements/service.test.js @@ -37,7 +37,7 @@ describe('entitlements service', () => { } })) jest.doMock('~/src/helpers/logging/logger.js', () => ({ - createLogger: jest.fn().mockReturnValue(mockLogger) + logger: mockLogger })) const fetchModule = await import('~/src/lib/fetch.js') diff --git a/src/api/forms/repositories/form-definition-repository.js b/src/api/forms/repositories/form-definition-repository.js index fa8ddfcc..d47f5a58 100644 --- a/src/api/forms/repositories/form-definition-repository.js +++ b/src/api/forms/repositories/form-definition-repository.js @@ -34,11 +34,9 @@ import { modifyUpdatePageFields, removeById } from '~/src/api/forms/repositories/helpers.js' -import { createLogger } from '~/src/helpers/logging/logger.js' +import { logger } from '~/src/helpers/logging/logger.js' import { DEFINITION_COLLECTION_NAME, db } from '~/src/mongo.js' -const logger = createLogger() - /** * Adds a form to the Form Store * @param {string} id - id diff --git a/src/api/forms/repositories/form-metadata-repository.js b/src/api/forms/repositories/form-metadata-repository.js index 2ca75e62..544142c9 100644 --- a/src/api/forms/repositories/form-metadata-repository.js +++ b/src/api/forms/repositories/form-metadata-repository.js @@ -15,13 +15,11 @@ import { MongoError, partialAuditFields } from '~/src/api/forms/service/shared.js' -import { createLogger } from '~/src/helpers/logging/logger.js' +import { logger } from '~/src/helpers/logging/logger.js' import { METADATA_COLLECTION_NAME, db } from '~/src/mongo.js' export const MAX_RESULTS = 100 -const logger = createLogger() - /** * Retrieves the list of documents from the database */ diff --git a/src/api/forms/repositories/form-metadata-repository.test.js b/src/api/forms/repositories/form-metadata-repository.test.js index d47122bd..5b7e5a18 100644 --- a/src/api/forms/repositories/form-metadata-repository.test.js +++ b/src/api/forms/repositories/form-metadata-repository.test.js @@ -70,10 +70,10 @@ jest.mock('~/src/mongo.js', () => { }) jest.mock('~/src/helpers/logging/logger.js', () => ({ - createLogger: () => ({ + logger: { error: jest.fn(), info: jest.fn() - }) + } })) describe('form-metadata-repository', () => { diff --git a/src/api/forms/repositories/form-versions-repository.js b/src/api/forms/repositories/form-versions-repository.js index 466cece4..0100fc9d 100644 --- a/src/api/forms/repositories/form-versions-repository.js +++ b/src/api/forms/repositories/form-versions-repository.js @@ -1,13 +1,11 @@ import { getErrorMessage } from '@defra/forms-model' import Boom from '@hapi/boom' -import { createLogger } from '~/src/helpers/logging/logger.js' +import { logger } from '~/src/helpers/logging/logger.js' import { VERSIONS_COLLECTION_NAME, db } from '~/src/mongo.js' export const MAX_VERSIONS = 100 -const logger = createLogger() - /** * Creates a new form version in the database * @param {FormVersionDocument} versionDocument - The form version document to create diff --git a/src/api/forms/repositories/helpers.js b/src/api/forms/repositories/helpers.js index 3edd9552..aa60b4fb 100644 --- a/src/api/forms/repositories/helpers.js +++ b/src/api/forms/repositories/helpers.js @@ -22,13 +22,11 @@ import * as formMetadataRepository from '~/src/api/forms/repositories/form-metad import * as formVersionsRepository from '~/src/api/forms/repositories/form-versions-repository.js' import { validate } from '~/src/api/forms/service/helpers/definition.js' import { repositionPaymentAndSummary } from '~/src/api/forms/service/migration-helpers.js' -import { createLogger } from '~/src/helpers/logging/logger.js' +import { logger } from '~/src/helpers/logging/logger.js' import { DEFINITION_COLLECTION_NAME, db } from '~/src/mongo.js' export const FORM_VERSION_METADATA_KEY = '$$__formVersion' -const logger = createLogger() - /** * Removes a row in a MongoDB collection by its unique ID and fail if not completed. * @param {ClientSession} session diff --git a/src/api/forms/repositories/secrets-repository.js b/src/api/forms/repositories/secrets-repository.js index dafcd47f..78e00429 100644 --- a/src/api/forms/repositories/secrets-repository.js +++ b/src/api/forms/repositories/secrets-repository.js @@ -2,13 +2,11 @@ import { getErrorMessage } from '@defra/forms-model' import Boom from '@hapi/boom' import { MongoServerError } from 'mongodb' -import { createLogger } from '~/src/helpers/logging/logger.js' +import { logger } from '~/src/helpers/logging/logger.js' import { SECRETS_COLLECTION_NAME, db } from '~/src/mongo.js' export const MAX_RESULTS = 100 -const logger = createLogger() - /** * Retrieves a form secret by form id and secret name * @param {string} formId - ID of the form diff --git a/src/api/forms/repositories/secrets-repository.test.js b/src/api/forms/repositories/secrets-repository.test.js index c9993802..b5e6959e 100644 --- a/src/api/forms/repositories/secrets-repository.test.js +++ b/src/api/forms/repositories/secrets-repository.test.js @@ -58,12 +58,12 @@ const mockLoggerInfo = jest.fn() const mockLoggerWarn = jest.fn() jest.mock('~/src/helpers/logging/logger.js', () => ({ - createLogger: jest.fn().mockReturnValue({ + logger: { // @ts-expect-error - error handling uses any type error: (err, message) => mockLoggerError(err, message), warn: () => mockLoggerWarn(), info: () => mockLoggerInfo() - }) + } })) describe('secrets-repository', () => { diff --git a/src/api/forms/service/component.js b/src/api/forms/service/component.js index f817821c..ee55aa61 100644 --- a/src/api/forms/service/component.js +++ b/src/api/forms/service/component.js @@ -10,7 +10,7 @@ import * as formMetadata from '~/src/api/forms/repositories/form-metadata-reposi import { findComponent } from '~/src/api/forms/repositories/helpers.js' import { getFormDefinition } from '~/src/api/forms/service/definition.js' import { getFormDefinitionPage } from '~/src/api/forms/service/page.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { publishFormUpdatedEvent } from '~/src/messaging/publish.js' import { client } from '~/src/mongo.js' diff --git a/src/api/forms/service/conditions.js b/src/api/forms/service/conditions.js index 58330bc8..5a93ecdf 100644 --- a/src/api/forms/service/conditions.js +++ b/src/api/forms/service/conditions.js @@ -2,7 +2,7 @@ import { FormDefinitionRequestType, getErrorMessage } from '@defra/forms-model' import * as formDefinition from '~/src/api/forms/repositories/form-definition-repository.js' import * as formMetadata from '~/src/api/forms/repositories/form-metadata-repository.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { publishFormUpdatedEvent } from '~/src/messaging/publish.js' import { client } from '~/src/mongo.js' diff --git a/src/api/forms/service/definition.js b/src/api/forms/service/definition.js index fc21070a..5d24e3ad 100644 --- a/src/api/forms/service/definition.js +++ b/src/api/forms/service/definition.js @@ -19,11 +19,8 @@ import { import { getValidationSchema } from '~/src/api/forms/service/helpers/definition.js' import { getForm } from '~/src/api/forms/service/index.js' import { existsFormSecret } from '~/src/api/forms/service/secrets.js' -import { - logger, - mapForm, - partialAuditFields -} from '~/src/api/forms/service/shared.js' +import { mapForm, partialAuditFields } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { publishDraftCreatedFromLiveEvent, publishFormDraftDeletedEvent, diff --git a/src/api/forms/service/helpers/definition.js b/src/api/forms/service/helpers/definition.js index 390d7dce..2bcdf944 100644 --- a/src/api/forms/service/helpers/definition.js +++ b/src/api/forms/service/helpers/definition.js @@ -9,7 +9,7 @@ import { import Joi from 'joi' import { InvalidFormDefinitionError } from '~/src/api/forms/errors.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' /** * Determines the correct validation schema based on the form definition's schema property diff --git a/src/api/forms/service/helpers/definition.test.js b/src/api/forms/service/helpers/definition.test.js index 96130288..10a60e30 100644 --- a/src/api/forms/service/helpers/definition.test.js +++ b/src/api/forms/service/helpers/definition.test.js @@ -20,6 +20,14 @@ import { validatePaymentAmount } from '~/src/api/forms/service/helpers/definition.js' +jest.mock('~/src/helpers/logging/logger.js', () => ({ + logger: { + info: jest.fn(), + warn: jest.fn(), + error: jest.fn() + } +})) + describe('definition helpers', () => { describe('getValidationSchema', () => { it('should return V1 schema when schema is V1', () => { diff --git a/src/api/forms/service/index.js b/src/api/forms/service/index.js index df70d4f5..72225851 100644 --- a/src/api/forms/service/index.js +++ b/src/api/forms/service/index.js @@ -16,7 +16,6 @@ import { } from '~/src/api/forms/service/helpers/definition.js' import { MongoError, - logger, mapForm, partialAuditFields } from '~/src/api/forms/service/shared.js' @@ -25,6 +24,7 @@ import { removeFormVersions } from '~/src/api/forms/service/versioning.js' import * as formTemplates from '~/src/api/forms/templates.js' +import { logger } from '~/src/helpers/logging/logger.js' import { getFormMetadataAuditMessages } from '~/src/messaging/mappers/form-events-bulk.js' import { bulkPublishEvents, diff --git a/src/api/forms/service/lists.js b/src/api/forms/service/lists.js index f2a45949..82c727fb 100644 --- a/src/api/forms/service/lists.js +++ b/src/api/forms/service/lists.js @@ -8,7 +8,7 @@ import Boom from '@hapi/boom' import * as formDefinition from '~/src/api/forms/repositories/form-definition-repository.js' import * as formMetadata from '~/src/api/forms/repositories/form-metadata-repository.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { publishFormUpdatedEvent } from '~/src/messaging/publish.js' import { saveToS3 } from '~/src/messaging/s3.js' import { client } from '~/src/mongo.js' diff --git a/src/api/forms/service/migration-helpers.test.js b/src/api/forms/service/migration-helpers.test.js index c8c1b3b7..a4f0e81d 100644 --- a/src/api/forms/service/migration-helpers.test.js +++ b/src/api/forms/service/migration-helpers.test.js @@ -55,6 +55,14 @@ jest.mock('~/src/api/forms/service/condition-migration-helpers.js', () => ({ isConditionData: jest.fn(() => true) })) +jest.mock('~/src/helpers/logging/logger.js', () => ({ + logger: { + info: jest.fn(), + warn: jest.fn(), + error: jest.fn() + } +})) + describe('migration helpers', () => { const summaryPageId = '449a45f6-4541-4a46-91bd-8b8931b07b50' diff --git a/src/api/forms/service/migration.js b/src/api/forms/service/migration.js index 6617260f..d725b353 100644 --- a/src/api/forms/service/migration.js +++ b/src/api/forms/service/migration.js @@ -8,7 +8,7 @@ import { import * as formDefinition from '~/src/api/forms/repositories/form-definition-repository.js' import * as formMetadata from '~/src/api/forms/repositories/form-metadata-repository.js' import { migrateToV2 } from '~/src/api/forms/service/migration-helpers.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { publishFormMigratedEvent } from '~/src/messaging/publish.js' import { client } from '~/src/mongo.js' /** diff --git a/src/api/forms/service/options.js b/src/api/forms/service/options.js index 9b3f0b02..81c668b8 100644 --- a/src/api/forms/service/options.js +++ b/src/api/forms/service/options.js @@ -2,7 +2,7 @@ import { FormDefinitionRequestType, getErrorMessage } from '@defra/forms-model' import * as formDefinition from '~/src/api/forms/repositories/form-definition-repository.js' import * as formMetadata from '~/src/api/forms/repositories/form-metadata-repository.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { publishFormUpdatedEvent } from '~/src/messaging/publish.js' import { client } from '~/src/mongo.js' diff --git a/src/api/forms/service/page.js b/src/api/forms/service/page.js index 08a089ab..cfb805ed 100644 --- a/src/api/forms/service/page.js +++ b/src/api/forms/service/page.js @@ -12,7 +12,8 @@ import { uniquePathGate } from '~/src/api/forms/repositories/helpers.js' import { getFormDefinition } from '~/src/api/forms/service/definition.js' -import { SUMMARY_PAGE_ID, logger } from '~/src/api/forms/service/shared.js' +import { SUMMARY_PAGE_ID } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { publishFormUpdatedEvent } from '~/src/messaging/publish.js' import { client } from '~/src/mongo.js' diff --git a/src/api/forms/service/report-overview.js b/src/api/forms/service/report-overview.js index ae345bf2..803bb0c6 100644 --- a/src/api/forms/service/report-overview.js +++ b/src/api/forms/service/report-overview.js @@ -13,7 +13,7 @@ import { StatusCodes } from 'http-status-codes' import * as formDefinition from '~/src/api/forms/repositories/form-definition-repository.js' import { getMetadataCursorOfAllForms } from '~/src/api/forms/repositories/form-metadata-repository.js' import { mapMetadata } from '~/src/api/forms/service/helpers/mapper.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { client } from '~/src/mongo.js' /** diff --git a/src/api/forms/service/report-timeline.js b/src/api/forms/service/report-timeline.js index 122c086a..9282f119 100644 --- a/src/api/forms/service/report-timeline.js +++ b/src/api/forms/service/report-timeline.js @@ -3,7 +3,7 @@ import { isSameDay } from 'date-fns' import { getMetadataCursorOfAllForms } from '~/src/api/forms/repositories/form-metadata-repository.js' import { mapMetadata } from '~/src/api/forms/service/helpers/mapper.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { client } from '~/src/mongo.js' /** diff --git a/src/api/forms/service/secrets.js b/src/api/forms/service/secrets.js index ddd59604..c8503afb 100644 --- a/src/api/forms/service/secrets.js +++ b/src/api/forms/service/secrets.js @@ -3,7 +3,7 @@ import { getErrorMessage } from '@defra/forms-model' import * as formMetadata from '~/src/api/forms/repositories/form-metadata-repository.js' import * as secretsRepository from '~/src/api/forms/repositories/secrets-repository.js' import { encryptSecret } from '~/src/api/forms/service/helpers/crypto.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { publishDeletedFormSecretEvent, publishSavedFormSecretEvent diff --git a/src/api/forms/service/sections.js b/src/api/forms/service/sections.js index 9acdd588..735512bc 100644 --- a/src/api/forms/service/sections.js +++ b/src/api/forms/service/sections.js @@ -2,7 +2,7 @@ import { getErrorMessage } from '@defra/forms-model' import * as formDefinition from '~/src/api/forms/repositories/form-definition-repository.js' import * as formMetadata from '~/src/api/forms/repositories/form-metadata-repository.js' -import { logger } from '~/src/api/forms/service/shared.js' +import { logger } from '~/src/helpers/logging/logger.js' import { publishFormUpdatedEvent } from '~/src/messaging/publish.js' import { client } from '~/src/mongo.js' diff --git a/src/api/forms/service/shared.js b/src/api/forms/service/shared.js index c8008567..e788941f 100644 --- a/src/api/forms/service/shared.js +++ b/src/api/forms/service/shared.js @@ -1,9 +1,6 @@ import { FormStatus } from '@defra/forms-model' import { ObjectId } from 'mongodb' -import { createLogger } from '~/src/helpers/logging/logger.js' - -export const logger = createLogger() export const defaultAuthor = { displayName: 'Unknown', id: '-1' diff --git a/src/api/forms/service/versioning.js b/src/api/forms/service/versioning.js index 3036f6d4..386d353e 100644 --- a/src/api/forms/service/versioning.js +++ b/src/api/forms/service/versioning.js @@ -7,11 +7,9 @@ import { allocateDraftVersion, stampFormVersion } from '~/src/api/forms/repositories/helpers.js' -import { createLogger } from '~/src/helpers/logging/logger.js' +import { logger } from '~/src/helpers/logging/logger.js' import { client } from '~/src/mongo.js' -const logger = createLogger() - /** * Wrapper function to instantiate new session and create form version * @param {string} formId diff --git a/src/helpers/fail-action.js b/src/helpers/fail-action.js index 29a10cc9..ac11bf6a 100644 --- a/src/helpers/fail-action.js +++ b/src/helpers/fail-action.js @@ -1,8 +1,6 @@ import { getErrorMessage } from '@defra/forms-model' -import { createLogger } from '~/src/helpers/logging/logger.js' - -const logger = createLogger() +import { logger } from '~/src/helpers/logging/logger.js' /** * Log and throw an error diff --git a/src/helpers/feedback-form/reinstate.test.js b/src/helpers/feedback-form/reinstate.test.js index 93f4db29..22c2879d 100644 --- a/src/helpers/feedback-form/reinstate.test.js +++ b/src/helpers/feedback-form/reinstate.test.js @@ -41,10 +41,10 @@ jest.mock('~/src/mongo.js', () => { jest.mock('~/src/api/forms/service/versioning.js') jest.mock('~/src/helpers/logging/logger.js', () => ({ - createLogger: () => ({ + logger: { error: jest.fn(), info: jest.fn() - }) + } })) describe('reinstate', () => { @@ -55,7 +55,7 @@ describe('reinstate', () => { .mockReturnValue(/** @type {any} */ (mockCollection)) jest.mocked(createFormVersion).mockResolvedValue(mockFormVersionDocument) jest.doMock('~/src/helpers/logging/logger.js', () => ({ - createLogger: jest.fn().mockReturnValue(mockLogger) + logger: mockLogger })) }) diff --git a/src/helpers/logging/logger.js b/src/helpers/logging/logger.js index 7e7e63f3..e71e12f3 100644 --- a/src/helpers/logging/logger.js +++ b/src/helpers/logging/logger.js @@ -5,6 +5,10 @@ import { loggerOptions } from '~/src/helpers/logging/logger-options.js' /** * Create a logger instance. */ -export function createLogger() { +function createPinoLogger() { return pino(loggerOptions) } + +// Singleton logger instance - pino adds 'exit' listeners to process, +// so we reuse a single instance to avoid MaxListenersExceededWarning +export const logger = createPinoLogger() diff --git a/src/index.js b/src/index.js index 0bfa9195..156aa743 100644 --- a/src/index.js +++ b/src/index.js @@ -2,9 +2,7 @@ import { chdir } from 'node:process' import { getErrorMessage } from '@defra/forms-model' -import { createLogger } from '~/src/helpers/logging/logger.js' - -const logger = createLogger() +import { logger } from '~/src/helpers/logging/logger.js' // Move working directory to build output chdir(import.meta.dirname) diff --git a/src/messaging/publish-base.js b/src/messaging/publish-base.js index a7ebaa68..03d49dd7 100644 --- a/src/messaging/publish-base.js +++ b/src/messaging/publish-base.js @@ -1,7 +1,7 @@ import { PublishCommand } from '@aws-sdk/client-sns' -import { logger } from '~/src/api/forms/service/shared.js' import { config } from '~/src/config/index.js' +import { logger } from '~/src/helpers/logging/logger.js' import { getSNSClient } from '~/src/messaging/sns.js' const snsTopicArn = config.get('snsTopicArn') diff --git a/src/messaging/publish-base.test.js b/src/messaging/publish-base.test.js index 976b9ae4..b30cc4b1 100644 --- a/src/messaging/publish-base.test.js +++ b/src/messaging/publish-base.test.js @@ -11,7 +11,13 @@ jest.mock('~/src/config/index.js', () => { awsRegion: 'eu-west-2', snsEndpoint: 'http://localhost', snsTopicArn: 'arn:aws:sns:eu-west-2:000000000000:forms_manager_events', - publishAuditEvents: true + publishAuditEvents: true, + log: { + enabled: true, + redact: ['req', 'res', 'responseTime'], + format: 'pino-pretty', + level: 'debug' + } } return { config: { diff --git a/src/plugins/auth/auth.test.js b/src/plugins/auth/auth.test.js index a9083633..f98f721c 100644 --- a/src/plugins/auth/auth.test.js +++ b/src/plugins/auth/auth.test.js @@ -3,11 +3,11 @@ const mockActualTestWarnFn = jest.fn() const mockActualTestInfoFn = jest.fn() jest.mock('~/src/helpers/logging/logger.js', () => ({ - createLogger: jest.fn().mockReturnValue({ + logger: { error: mockActualTestErrorFn, warn: mockActualTestWarnFn, info: mockActualTestInfoFn - }) + } })) jest.mock('~/src/config/index.js', () => ({ diff --git a/src/plugins/auth/index.js b/src/plugins/auth/index.js index e4e37051..4d91ea43 100644 --- a/src/plugins/auth/index.js +++ b/src/plugins/auth/index.js @@ -2,14 +2,12 @@ import Jwt from '@hapi/jwt' import { getUserScopes } from '~/src/api/entitlements/service.js' import { config } from '~/src/config/index.js' -import { createLogger } from '~/src/helpers/logging/logger.js' +import { logger } from '~/src/helpers/logging/logger.js' const oidcJwksUri = config.get('oidcJwksUri') const oidcVerifyAud = config.get('oidcVerifyAud') const oidcVerifyIss = config.get('oidcVerifyIss') -const logger = createLogger() - /** * Validates user credentials from JWT token * @param {Artifacts} artifacts - JWT artifacts diff --git a/src/server.js b/src/server.js index f7aba040..801ad860 100644 --- a/src/server.js +++ b/src/server.js @@ -2,9 +2,7 @@ import { getErrorMessage } from '@defra/forms-model' import { createServer } from '~/src/api/server.js' import { config } from '~/src/config/index.js' -import { createLogger } from '~/src/helpers/logging/logger.js' - -const logger = createLogger() +import { logger } from '~/src/helpers/logging/logger.js' process.on('unhandledRejection', (err) => { logger.error(