Skip to content

Commit

Permalink
fix: use pino-pretty if development OR filepath not supplied
Browse files Browse the repository at this point in the history
  • Loading branch information
meenahoda committed Nov 30, 2021
1 parent 4a3533f commit 5f74aa5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion lib/plugin/components/services/logger/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ class Logger {
this.loggerOutputFilePath = generateFilePath(process.env.LOGGER_OUTPUT_DIR_PATH)
options.messages.info(this.loggerOutputFilePath ? `Output logs to ${this.loggerOutputFilePath}` : 'No output file')
this.logger = this.createLogger()
this.logger.info('Logger initialised')
}

addLogger (namespace = 'logger') {
Expand Down Expand Up @@ -83,7 +84,7 @@ function generateFilePath (dirpath) {
function createMultistream (filepath, level) {
const streams = []
if (filepath) streams.push({ stream: pino.destination(filepath), level })
if (process.env.NODE_ENV === 'development') streams.push({ stream: pinoPretty() })
if (process.env.NODE_ENV === 'development' || !filepath) streams.push({ stream: pinoPretty() })
return pino.multistream(streams)
}

Expand Down

0 comments on commit 5f74aa5

Please sign in to comment.