Skip to content

Commit

Permalink
fix(gateway): Fixes audit warnings
Browse files Browse the repository at this point in the history
* Replaces express-pino-logger with pino-http

* Remove @types/yaml as not required
  • Loading branch information
phantomjinx committed Jun 28, 2024
1 parent cc06919 commit 51b306b
Show file tree
Hide file tree
Showing 3 changed files with 63 additions and 168 deletions.
4 changes: 1 addition & 3 deletions docker/gateway/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@
"dependencies": {
"cors": "^2.8.5",
"express": "^4.19.2",
"express-pino-logger": "^7.0.0",
"fetch-intercept": "^2.4.0",
"helmet": "^6.0.0",
"http-proxy-middleware": "^3.0.0",
Expand All @@ -36,22 +35,21 @@
"jwt-decode": "^4.0.0",
"method-override": "^3.0.0",
"pino": "^8.6.1",
"pino-http": "^10.1.0",
"pino-pretty": "^11.2.1",
"ts-node": "^10.9.2",
"yaml": "^2.4.3"
},
"devDependencies": {
"@types/cors": "^2.8.17",
"@types/express": "^4.17.21",
"@types/express-pino-logger": "^4.0.5",
"@types/jest": "^29.5.12",
"@types/jquery": "^3.5.29",
"@types/js-yaml": "^4.0.9",
"@types/method-override": "^0.0.35",
"@types/node": "^18.19.31",
"@types/pino-multi-stream": "^5.1.6",
"@types/supertest": "^6.0.2",
"@types/yaml": "^1.9.7",
"babel-jest": "^29.6.1",
"compression-webpack-plugin": "^11.1.0",
"copy-webpack-plugin": "^12.0.2",
Expand Down
8 changes: 5 additions & 3 deletions docker/gateway/src/logger.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
import pino from 'pino'
import expressPinoLogger from 'express-pino-logger'
import pinoHttpLogger from 'pino-http'

export const logger = pino({ level: process.env.LOG_LEVEL || 'info' })
export const expressLogger = expressPinoLogger(logger)
const level = process.env.LOG_LEVEL || 'info'

export const logger = pino({ level: level})
export const expressLogger = pinoHttpLogger({ logger: logger })
Loading

0 comments on commit 51b306b

Please sign in to comment.