decentraland / comms-gatekeeper / 27881547914

85%

main: 85%

Pull #266

github

refactor: replace WKC request logger with a native @dcl/http-server middleware

The request logger was the last server-instrumentation helper still pulling in a
@well-known-components server type, forcing an `as unknown as IHttpServerComponent`
cast on the native @dcl/http-server server.

Replace @well-known-components/http-requests-logger-component with a small native
middleware (src/logic/http-requests-logger.ts) that runs directly on the
@dcl/http-server server. The log format and skipped health paths are preserved:
`[METHOD: /path?query#hash]` in and `[METHOD: /path?query#hash][status]` out,
skipping /health/live and /health/ready.

This drops the dependency and removes the last WKC server-type bridge from
components.ts. All server instrumentation (tracer, metrics, request logger) now runs
on the native server with no casts.

1061 of 1372 branches covered (77.33%)

Branch coverage included in aggregate %.

44 of 46 new or added lines in 6 files covered. (95.65%)

2986 of 3377 relevant lines covered (88.42%)

77.14 hits per line