Skip to content

Commit

Permalink
Log x-routes usage
Browse files Browse the repository at this point in the history
For incoming requests, log the number of endpoints in the x-routes
header. If no x-routes is present, log as "num_x_routes": 0
  • Loading branch information
joodie committed Jan 8, 2025
1 parent 9d35895 commit 261016b
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions policies/lifecycle-logger/logger.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
*/

const jsonLog = require('../../lib/json_log')
const xroute = require('../../lib/xroute')
const ensureTraceParent = require('../../lib/ensure_traceparent')

module.exports = () => {
Expand All @@ -23,12 +24,12 @@ module.exports = () => {
const reqTimerStart = new Date()
const method = req.method
const url = req.originalUrl
const routes = xroute.decode(req.headers['x-route'])

res.on('finish', () => {
const app = req.egContext.app // set by gatekeeper policy
const reqTimerEnd = new Date()
const statusCode = res.statusCode

const infoProps = {
side: 'client',
short_message: `${req.traceparent.traceId} - ${method} ${url} ${statusCode}`,
Expand All @@ -40,7 +41,8 @@ module.exports = () => {
http_status: statusCode,
request_method: method,
url,
time_ms: reqTimerEnd - reqTimerStart
time_ms: reqTimerEnd - reqTimerStart,
num_x_routes: routes ? routes.length : 0
}

if (res.error_msg) {
Expand Down

0 comments on commit 261016b

Please sign in to comment.