@saidsef/tracing-node 4.0.0 → 4.1.0

package/README.md

@@ -15,6 +15,7 @@ Effortlessly supercharge your applications with world-class distributed tracing!
 | Feature | Description |
 |---------|-------------|
 | HTTP/HTTPS instrumentation | Automatic service detection |
+| fetch/undici instrumentation | Outgoing `globalThis.fetch` calls |
 | Express.js support | Framework instrumentation |
 | Elasticsearch client | Database instrumentation |
 | IORedis client | Cache instrumentation |

package/libs/index.mjs

@@ -25,6 +25,7 @@ import {ExpressInstrumentation} from '@opentelemetry/instrumentation-express';
 import {NodeTracerProvider} from '@opentelemetry/sdk-trace-node';
 import {OTLPTraceExporter} from '@opentelemetry/exporter-trace-otlp-grpc';
 import {PinoInstrumentation} from '@opentelemetry/instrumentation-pino';
+import {UndiciInstrumentation} from '@opentelemetry/instrumentation-undici';
 import {IORedisInstrumentation} from '@opentelemetry/instrumentation-ioredis';
 import {registerInstrumentations} from '@opentelemetry/instrumentation';
 import {FsInstrumentation} from '@opentelemetry/instrumentation-fs';
@@ -50,6 +51,21 @@ const truncateArg = (value, limit) => {
   return str.length > limit ? `${str.substring(0, limit)}...` : str;
 };
 
+// Tempo names a service-graph node from peer.service, which no instrumentation
+// emits, so both the http and undici hooks below derive it from the remote host.
+const PEER_SERVICES = ['elasticsearch', 'redis'];
+
+const setPeerService = (span, host) => {
+  if (!host) return;
+  for (const service of PEER_SERVICES) {
+    if (host.includes(service)) {
+      span.setAttribute('peer.service', service);
+      span.setAttribute('db.system.name', service);
+      return;
+    }
+  }
+};
+
 let tracerProvider = null; // Declare provider in module scope for access in stopTracing
 
 /**
@@ -134,18 +150,7 @@ export function setupTracing(options = {}) {
 
   // Only an outgoing ClientRequest carries .host, so bailing without it keeps
   // server spans out: peer.service must name the remote service being called.
-  const applyCustomAttributesOnSpan = (span, request) => {
-    const host = request?.host;
-    if (!host) return;
-
-    for (const service of ['elasticsearch', 'redis']) {
-      if (host.includes(service)) {
-        span.setAttribute('peer.service', service);
-        span.setAttribute('db.system.name', service);
-        return;
-      }
-    }
-  };
+  const applyCustomAttributesOnSpan = (span, request) => setPeerService(span, request?.host);
 
   const instrumentations = [
     new HttpInstrumentation({
@@ -178,6 +183,14 @@ export function setupTracing(options = {}) {
         if (requestId) span.setAttribute('http.request_id', requestId);
       },
     }),
+    // globalThis.fetch runs on undici, which never touches the http/https
+    // modules HttpInstrumentation patches. Without this, fetch calls produce no
+    // client span and inject no traceparent, so the callee starts a new trace
+    // and the two services can never be paired into a service-graph edge.
+    new UndiciInstrumentation({
+      // UndiciRequest exposes origin, not the .host the http hook reads.
+      requestHook: (span, request) => setPeerService(span, request?.origin),
+    }),
     new ExpressInstrumentation({
       requestHook: (span, info) => {
         // info is ExpressRequestInfo: { request, route, layerType }

package/libs/index.test.mjs

@@ -49,6 +49,20 @@ describe('setupTracing', () => {
     assert.ok(tracer, 'tracer should be defined');
   });
 
+  // globalThis.fetch runs on undici, so it is invisible to HttpInstrumentation.
+  // Registering it must not disturb setup, and the patch has to land on the
+  // global fetch itself - otherwise outgoing calls carry no traceparent.
+  it('should instrument global fetch', () => {
+    const before = globalThis.fetch;
+    const tracer = setupTracing({
+      serviceName: 'test-service',
+      url: 'http://localhost:4317',
+    });
+    assert.ok(tracer, 'tracer should be defined');
+    assert.strictEqual(typeof globalThis.fetch, 'function', 'global fetch should still be callable');
+    assert.ok(before, 'global fetch should exist on a supported runtime');
+  });
+
   it('should accept optional instrumentations', () => {
     const tracer = setupTracing({
       serviceName: 'test-service',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saidsef/tracing-node",
-  "version": "4.0.0",
+  "version": "4.1.0",
   "description": "tracing NodeJS - Wrapper for OpenTelemetry instrumentation packages",
   "main": "libs/index.mjs",
   "scripts": {
@@ -23,7 +23,7 @@
   "author": "Said Sef <saidsef@gmail.com>",
   "license": "Apache-2.0",
   "engines": {
-    "node": ">= 20"
+    "node": ">= 20.6.0"
   },
   "bugs": {
     "url": "https://github.com/saidsef/tracing-node/issues"
@@ -41,6 +41,7 @@
     "@opentelemetry/instrumentation-http": "^0.221.0",
     "@opentelemetry/instrumentation-ioredis": "^0.69.0",
     "@opentelemetry/instrumentation-pino": "^0.67.0",
+    "@opentelemetry/instrumentation-undici": "^0.31.0",
     "@opentelemetry/resources": "^2.10.0",
     "@opentelemetry/sdk-trace-base": "^2.10.0",
     "@opentelemetry/sdk-trace-node": "^2.10.0",