From f85ecd481062fcd3dd530221108f86d672a72f80 Mon Sep 17 00:00:00 2001
From: Rory Powell <rory.codes@gmail.com>
Date: Thu, 12 Jan 2023 10:25:21 +0000
Subject: [PATCH] Fix pino request id generation (#9324)

Remove the duplicated pino config in favour of the common config. Move request id generation into common configuration
---
 packages/backend-core/src/logging.ts |  1 +
 packages/backend-core/src/pino.ts    | 14 --------------
 2 files changed, 1 insertion(+), 14 deletions(-)
 delete mode 100644 packages/backend-core/src/pino.ts

diff --git a/packages/backend-core/src/logging.ts b/packages/backend-core/src/logging.ts
index 711c38917c..8f2ae6e619 100644
--- a/packages/backend-core/src/logging.ts
+++ b/packages/backend-core/src/logging.ts
@@ -40,6 +40,7 @@ export function pinoSettings(): Options {
     prettyPrint: {
       levelFirst: true,
     },
+    genReqId: correlator.getId,
     level: env.LOG_LEVEL || "error",
     autoLogging: {
       ignore: (req: IncomingMessage) => !!req.url?.includes("/health"),
diff --git a/packages/backend-core/src/pino.ts b/packages/backend-core/src/pino.ts
deleted file mode 100644
index 415aebff5b..0000000000
--- a/packages/backend-core/src/pino.ts
+++ /dev/null
@@ -1,14 +0,0 @@
-const env = require("./environment")
-import { Options } from "pino-http"
-const correlator = require("correlation-id")
-
-export const pinoSettings = (): Options => ({
-  prettyPrint: {
-    levelFirst: true,
-  },
-  genReqId: correlator.getId,
-  level: env.LOG_LEVEL || "error",
-  autoLogging: {
-    ignore: (req: any) => req.url.includes("/health"),
-  },
-})