@naturalcycles/backend-lib 9.85.1 → 9.86.0

package/dist/server/trace.util.js

@@ -10,12 +10,11 @@ const traceIdByRequest = new WeakMap();
  * every log line.
  */
 export function getRequestTraceId(req) {
-    let traceId = traceIdByRequest.get(req);
-    if (traceId === undefined) {
-        traceId = getRequestTraceContext(req)?.traceId ?? null;
-        traceIdByRequest.set(req, traceId);
-    }
-    return traceId ?? undefined;
+    return traceIdByRequest.getOrInsertComputed(req, getRequestTraceIdOrNull) ?? undefined;
+}
+// Passed by reference, so the per-log-line call allocates no closure
+function getRequestTraceIdOrNull(req) {
+    return getRequestTraceContext(req)?.traceId ?? null;
 }
 export function getRequestTraceContext(req) {
     // Reads from `req.headers` and not `req.header()`, so a bare request object (as in tests) works too

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@naturalcycles/backend-lib",
   "type": "module",
-  "version": "9.85.1",
+  "version": "9.86.0",
   "dependencies": {
     "@naturalcycles/db-lib": "^10",
     "@naturalcycles/js-lib": "^15",

package/src/server/trace.util.ts

@@ -23,12 +23,12 @@ const traceIdByRequest = new WeakMap<BackendRequest, string | null>()
  * every log line.
  */
 export function getRequestTraceId(req: BackendRequest): string | undefined {
-  let traceId = traceIdByRequest.get(req)
-  if (traceId === undefined) {
-    traceId = getRequestTraceContext(req)?.traceId ?? null
-    traceIdByRequest.set(req, traceId)
-  }
-  return traceId ?? undefined
+  return traceIdByRequest.getOrInsertComputed(req, getRequestTraceIdOrNull) ?? undefined
+}
+
+// Passed by reference, so the per-log-line call allocates no closure
+function getRequestTraceIdOrNull(req: BackendRequest): string | null {
+  return getRequestTraceContext(req)?.traceId ?? null
 }
 
 export function getRequestTraceContext(req: BackendRequest): TraceContext | undefined {