fix: log error instead of warn on 500s

hapijs · Apr 22, 2019 · 7823a6d · 7823a6d
1 parent d50a867
commit 7823a6d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -80,7 +80,7 @@ async function register (server, options) {
 
   server.events.on('log', function (event) {
     if (event.error) {
-      logger.warn({ err: event.error })
+      logger.error({ err: event.error })
     } else {
       logEvent(logger, event)
     }
@@ -96,7 +96,7 @@ async function register (server, options) {
     request.logger = request.logger || logger.child({ req: request })
 
     if (event.error && isEnabledLogEvent(options, 'request-error')) {
-      request.logger.warn({
+      request.logger.error({
         err: event.error
       }, 'request error')
     } else if (event.channel === 'app') {

diff --git a/test.js b/test.js
@@ -255,7 +255,7 @@ experiment('logs each request', () => {
     await registerWithSink(server, 'info', (data, enc, cb) => {
       if (count === 0) {
         expect(data.err.message).to.equal('boom')
-        expect(data.level).to.equal(40)
+        expect(data.level).to.equal(50)
         expect(data.msg).to.equal('request error')
       } else {
         expect(data.res.statusCode).to.equal(500)
@@ -381,7 +381,7 @@ experiment('logs through server.log', () => {
       expect(data.err.message).to.equal('foobar')
       expect(data.err.stack).to.exist()
       // highest level tag
-      expect(data.level).to.equal(40)
+      expect(data.level).to.equal(50)
       resolver()
     })