@dotcom-reliability-kit/logger
Advanced tools
Comparing version
@@ -72,7 +72,10 @@ const pino = require('pino').default; | ||
* prettification if the `NODE_ENV` environment variable is | ||
* set to "production". | ||
* set to "production". We also disallow prettification if the | ||
* cloud provider is AWS, pino-pretty does not work well with | ||
* CloudWatch logs | ||
* | ||
* @type {boolean} | ||
*/ | ||
const prettificationAllowed = appInfo.environment !== 'production'; | ||
const prettificationAllowed = | ||
appInfo.environment !== 'production' && appInfo.cloudProvider !== 'aws'; | ||
@@ -79,0 +82,0 @@ /** |
{ | ||
"name": "@dotcom-reliability-kit/logger", | ||
"version": "4.0.0", | ||
"version": "4.0.1", | ||
"description": "A simple and fast logger based on Pino, with FT preferences baked in", | ||
@@ -19,3 +19,3 @@ "repository": { | ||
"dependencies": { | ||
"@dotcom-reliability-kit/app-info": "^4.0.0", | ||
"@dotcom-reliability-kit/app-info": "^4.1.0", | ||
"@dotcom-reliability-kit/serialize-error": "^4.0.0", | ||
@@ -22,0 +22,0 @@ "lodash.clonedeep": "^4.5.0", |
@@ -606,3 +606,3 @@ | ||
Using `@dotcom-reliability-kit/logger` in production requires that your application can handle logs to `stdout` and sends these logs to somewhere more permanent. On Heroku this means you're required to have [migrated to Log Drains](https://financialtimes.atlassian.net/wiki/spaces/DS/pages/7883555001/Migrating+an+app+to+Heroku+log+drains). On AWS Lambda it means you must be sending logs to CloudWatch ([see tech hub documentation](https://tech.in.ft.com/tech-topics/logging/amazon-cloudwatch-logs#cloudformation-code-for-forwarding-lambda-logs-to-splunk)). | ||
Using `@dotcom-reliability-kit/logger` in production requires that your application can handle logs to `stdout` and sends these logs to somewhere more permanent. On Heroku this means you're required to have [migrated to Log Drains](https://financialtimes.atlassian.net/wiki/spaces/DS/pages/7883555001/Migrating+an+app+to+Heroku+log+drains). On AWS Lambda it means you must be sending logs to CloudWatch ([see tech hub documentation](https://tech.in.ft.com/tech-topics/observability/amazon-cloudwatch-logs#cloudformation-code-for-forwarding-lambda-logs-to-splunk)). | ||
@@ -609,0 +609,0 @@ ### Testing |
46316
0.35%729
0.41%