From cfe3db835b54b950783594d88fefc3734e4d03f9 Mon Sep 17 00:00:00 2001
From: mike12345567 <me@michaeldrury.co.uk>
Date: Thu, 4 Aug 2022 20:23:45 +0100
Subject: [PATCH] Updating pino logging to remove health check spam.

---
 packages/backend-core/src/environment.ts |  1 +
 packages/backend-core/src/index.ts       |  2 ++
 packages/backend-core/src/pino.js        | 11 +++++++++++
 packages/server/src/app.ts               | 10 ++--------
 packages/worker/src/index.ts             | 11 ++---------
 5 files changed, 18 insertions(+), 17 deletions(-)
 create mode 100644 packages/backend-core/src/pino.js

diff --git a/packages/backend-core/src/environment.ts b/packages/backend-core/src/environment.ts
index 37804b31a6..51cc721ded 100644
--- a/packages/backend-core/src/environment.ts
+++ b/packages/backend-core/src/environment.ts
@@ -55,6 +55,7 @@ const env = {
   DEFAULT_LICENSE: process.env.DEFAULT_LICENSE,
   SERVICE: process.env.SERVICE || "budibase",
   MEMORY_LEAK_CHECK: process.env.MEMORY_LEAK_CHECK || false,
+  LOG_LEVEL: process.env.LOG_LEVEL,
   DEPLOYMENT_ENVIRONMENT:
     process.env.DEPLOYMENT_ENVIRONMENT || "docker-compose",
   _set(key: any, value: any) {
diff --git a/packages/backend-core/src/index.ts b/packages/backend-core/src/index.ts
index 35777ae817..ced4630fb7 100644
--- a/packages/backend-core/src/index.ts
+++ b/packages/backend-core/src/index.ts
@@ -15,6 +15,7 @@ import auth from "./auth"
 import constants from "./constants"
 import * as dbConstants from "./db/constants"
 import logging from "./logging"
+import pino from "./pino"
 
 // mimic the outer package exports
 import * as db from "./pkg/db"
@@ -53,6 +54,7 @@ const core = {
   errors,
   logging,
   roles,
+  ...pino,
   ...errorClasses,
 }
 
diff --git a/packages/backend-core/src/pino.js b/packages/backend-core/src/pino.js
new file mode 100644
index 0000000000..69962b3841
--- /dev/null
+++ b/packages/backend-core/src/pino.js
@@ -0,0 +1,11 @@
+const env = require("./environment")
+
+exports.pinoSettings = () => ({
+  prettyPrint: {
+    levelFirst: true,
+  },
+  level: env.LOG_LEVEL || "error",
+  autoLogging: {
+    ignore: req => req.url.includes("/health"),
+  },
+})
diff --git a/packages/server/src/app.ts b/packages/server/src/app.ts
index 32951cc47e..62301d57ca 100644
--- a/packages/server/src/app.ts
+++ b/packages/server/src/app.ts
@@ -15,6 +15,7 @@ const Sentry = require("@sentry/node")
 const fileSystem = require("./utilities/fileSystem")
 const bullboard = require("./automations/bullboard")
 const { logAlert } = require("@budibase/backend-core/logging")
+const { pinoSettings } = require("@budibase/backend-core")
 const { Thread } = require("./threads")
 import redis from "./utilities/redis"
 import * as migrations from "./migrations"
@@ -35,14 +36,7 @@ app.use(
   })
 )
 
-app.use(
-  pino({
-    prettyPrint: {
-      levelFirst: true,
-    },
-    level: env.LOG_LEVEL || "error",
-  })
-)
+app.use(pino(pinoSettings()))
 
 if (!env.isTest()) {
   const plugin = bullboard.init()
diff --git a/packages/worker/src/index.ts b/packages/worker/src/index.ts
index f8031abacb..6fb954a1b5 100644
--- a/packages/worker/src/index.ts
+++ b/packages/worker/src/index.ts
@@ -18,7 +18,7 @@ const http = require("http")
 const api = require("./api")
 const redis = require("./utilities/redis")
 const Sentry = require("@sentry/node")
-import { events } from "@budibase/backend-core"
+import { events, pinoSettings } from "@budibase/backend-core"
 
 // this will setup http and https proxies form env variables
 bootstrap()
@@ -30,14 +30,7 @@ app.keys = ["secret", "key"]
 // set up top level koa middleware
 app.use(koaBody({ multipart: true }))
 app.use(koaSession(app))
-app.use(
-  logger({
-    prettyPrint: {
-      levelFirst: true,
-    },
-    level: env.LOG_LEVEL || "error",
-  })
-)
+app.use(logger(pinoSettings()))
 
 // authentication
 app.use(passport.initialize())