fix(observability): rename x-request-id log field to x_request_id (#189)

Co-authored-by: James Willis <james.willis@elasticpath.com>

Author: jameswillis99

platform/wab/src/wab/server/AppServer.ts

@@ -456,7 +456,7 @@ export function addLoggingMiddleware(app: express.Application) {
           requestMethod: req.method,
           requestOriginalUrl: req.originalUrl,
           requestStatusCode: res.statusCode,
-          "x-request-id": req.id,
+          x_request_id: req.id,
           remoteAddr: req.ip,
           userEmail: req.user?.email,
           referrer: req.get("referrer"),

platform/wab/src/wab/server/observability/PinoLogger.ts

@@ -73,7 +73,7 @@ export class PinoLogger implements Logger {
     const { requestId } = requestStorage.getStore() ?? {};
     const logEntry = {
       message,
-      ...(requestId ? { "x-request-id": requestId } : {}),
+      ...(requestId ? { x_request_id: requestId } : {}),
       ...payload,
     };