BuffLog middleware INFRA-382

[erickhun]

In order to replace the previous @bufferapp/middleware

• Add getLogger to be able to use the pino instance
• Ddd the middleware() function (use pino-http under the hood)

Usage:

const app = express();
app.use(BuffLog.middleware())

or if we want to use the logger instance instead to personalise how the middleware works :

const app = express();
const middlewareLogger = require('pino-http')({
  logger: bufflog.getLogger(),
 // ... options
})
app.use(middlewareLogger)

the output looks like this:

{
    "dd": {
        "span_id": "6047504131036872311",
        "trace_id": "6047504131036872311"
    },
    "level": 200,
    "message": "request completed",
    "req": {
        "headers": {
            "accept": "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9",
            "accept-encoding": "gzip, deflate, br",
            "accept-language": "en-US,en;q=0.9,fr;q=0.8,zh-TW;q=0.7,zh;q=0.6",
            "cache-control": "max-age=0",
            "connection": "keep-alive",
            "host": "localhost:4000",
            "if-none-match": "W/\"11-IkjuL6CqqtmReFMfkkvwC0sKj04\"",
            "sec-fetch-dest": "document",
            "sec-fetch-mode": "navigate",
            "sec-fetch-site": "cross-site",
            "sec-fetch-user": "?1",
            "upgrade-insecure-requests": "1",
            "user-agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.149 Safari/537.36"
        },
        "id": 1,
        "method": "GET",
        "remoteAddress": "::1",
        "remotePort": 61841,
        "url": "/"
    },
    "res": {
        "headers": {
            "etag": "W/\"11-IkjuL6CqqtmReFMfkkvwC0sKj04\"",
            "x-powered-by": "Express"
        },
        "statusCode": 304
    },
    "responseTime": 10,
    "time": 1586254897546,
    "v": 1
}

There was also many stats calculated in the previous loggers. I feel like those should be aimed to be in the APM tracking instead of the logs.

@esclapes up for a review?

[esclapes]

This is great @erickhun I have suggested a small change for extra flexibility (when needed), but this looks good to me! 🌮 🌮

[colinscape]

Nice, this looks great to me! 👍