Merge pull request #29 from pinojs/working-pretty-print-again

Restored prettyPrint support
hapijs · Nov 13, 2017 · 8715e64 · 8715e64
2 parents 52f4efb + ee2ba18
commit 8715e64
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/example.js b/example.js
@@ -33,10 +33,7 @@ async function start () {
     }
   })
 
-  // the logger is available in server.app
-  server.app.logger.warn('Pino is registered')
-
-  // also as a decorated API
+  // as a decorated API
   server.logger().info('another way for accessing it')
 
   // and through Hapi standard logging system

diff --git a/index.js b/index.js
@@ -30,6 +30,9 @@ async function register (server, options) {
     var stream = options.stream || process.stdout
 
     if (options.prettyPrint) {
+      // pino has a similar logic that works slightly different
+      // we must disable that
+      delete options.prettyPrint
       var pretty = pino.pretty()
       pretty.pipe(stream)
       stream = pretty

diff --git a/package.json b/package.json
@@ -21,15 +21,15 @@
     "code": "^5.1.2",
     "coveralls": "^3.0.0",
     "flush-write-stream": "^1.0.0",
-    "hapi": "^17.0.0-rc10",
+    "hapi": "^17.0.0",
     "lab": "^15.0.0",
     "make-promises-safe": "^1.1.0",
     "pre-commit": "^1.1.2",
     "split2": "^2.2.0",
     "standard": "^10.0.3"
   },
   "dependencies": {
-    "pino": "^4.9.0"
+    "pino": "^4.10.0"
   },
   "repository": {
     "type": "git",