Skip to content
This repository was archived by the owner on Aug 12, 2023. It is now read-only.

Commit a3dcaf1

Browse files
authored
Stop logging HTTP requests via pino (#350)
1 parent 9527ffa commit a3dcaf1

4 files changed

Lines changed: 19 additions & 83 deletions

File tree

package-lock.json

Lines changed: 10 additions & 71 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@
3737
"koa-compress": "3.0.0",
3838
"koa-etag": "3.0.0",
3939
"koa-helmet": "5.2.0",
40-
"koa-pino-logger": "2.1.3",
4140
"koa-router": "7.4.0",
4241
"lodash": "4.17.15",
4342
"memory-cache": "0.2.0",

src/app/start.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,18 +3,17 @@ const etag = require('koa-etag');
33
const helmet = require('koa-helmet');
44
const Koa = require('koa');
55

6+
const { getLogger } = require('../util/logging');
67
const errorLogger = require('../util/error-logger');
7-
const logging = require('../util/logging');
88
const middleware = require('./middleware');
99
const routes = require('./routes');
1010

1111
const start = port => {
12-
const logger = logging.getLogger('application');
12+
const logger = getLogger('application');
1313
const app = new Koa();
1414

1515
errorLogger.attachToApp(app);
1616

17-
app.use(logging.getMiddleware());
1817
app.use(middleware.error());
1918
app.use(helmet());
2019
app.use(middleware.cors());

src/util/logging.js

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ const _ = require('lodash');
22
const os = require('os');
33
const pino = require('pino');
44
const pinoElastic = require('pino-elasticsearch');
5-
const pinoKoa = require('koa-pino-logger');
65

76
let logger;
87
let middleware;
@@ -30,13 +29,13 @@ const init = config => {
3029
type: 'log',
3130
});
3231

33-
const pinoOptions = {
34-
base: { group: 'application', pid: process.pid, hostname: os.hostname() },
35-
level: 'info',
36-
};
37-
38-
logger = pino(pinoOptions, streamToElasticsearch);
39-
middleware = pinoKoa(pinoOptions, streamToElasticsearch);
32+
logger = pino(
33+
{
34+
base: { group: 'application', pid: process.pid, hostname: os.hostname() },
35+
level: 'info',
36+
},
37+
streamToElasticsearch,
38+
);
4039
};
4140

4241
const getLogger = logGroup => {

0 commit comments

Comments
 (0)