[express-pino-logger]: add destination argument type (#38153)

This commit is contained in:
iamolegga 2019-09-06 23:36:55 +03:00 committed by Andrew Casey
parent 9fa230cc66
commit 571d84405e
2 changed files with 20 additions and 5 deletions

View File

@ -4,18 +4,31 @@ import expressPinoLogger = require('express-pino-logger');
const server = express();
// no options
// no arguments
let middleware = expressPinoLogger();
server.use(middleware);
// pino own options
// pino options only
const pinoOptions: pino.LoggerOptions = {};
middleware = expressPinoLogger(pinoOptions);
server.use(middleware);
// options with existing logger
// pino destination only
const pinoDestination: pino.DestinationStream = pino.destination('/log/path');
middleware = expressPinoLogger(pinoDestination);
server.use(middleware);
// pino options and destination
const pinoOpts: pino.LoggerOptions = {};
const pinoDest: pino.DestinationStream = pino.destination('/log/path');
middleware = expressPinoLogger(pinoOpts, pinoDest);
server.use(middleware);
// existing logger
const logger = pino();
const optionsWithLogger = { logger };

View File

@ -5,9 +5,11 @@
// TypeScript Version: 2.7
import { Handler } from 'express';
import { LoggerOptions, Logger } from 'pino';
import { DestinationStream, LoggerOptions, Logger } from 'pino';
declare function expressPinoLogger(options?: LoggerOptions | { logger?: Logger }): Handler;
declare function expressPinoLogger(optionsOrStream?: LoggerOptions | DestinationStream | { logger: Logger }): Handler;
declare function expressPinoLogger(options: LoggerOptions, stream: DestinationStream): Handler;
export = expressPinoLogger;