fix(instrumentation-undici): fix a possible crash if the request path is a full URL (#2518)

Closes: #2471

Author: trentm

plugins/node/instrumentation-undici/src/undici.ts

@@ -188,7 +188,14 @@ export class UndiciInstrumentation extends InstrumentationBase<UndiciInstrumenta
     }
 
     const startTime = hrTime();
-    const requestUrl = new URL(request.origin + request.path);
+    let requestUrl;
+    try {
+      requestUrl = new URL(request.path, request.origin);
+    } catch (err) {
+      this._diag.warn('could not determine url.full:', err);
+      // Skip instrumenting this request.
+      return;
+    }
     const urlScheme = requestUrl.protocol.replace(':', '');
     const requestMethod = this.getRequestMethod(request.method);
     const attributes: Attributes = {

plugins/node/instrumentation-undici/test/undici.test.ts

@@ -831,5 +831,33 @@ describe('UndiciInstrumentation `undici` tests', function () {
         'user-agent is undefined'
       );
     });
+
+    it('should create valid span if request.path is a full URL', async function () {
+      let spans = memoryExporter.getFinishedSpans();
+      assert.strictEqual(spans.length, 0);
+
+      const origin = `${protocol}://${hostname}:${mockServer.port}`;
+      const fullUrl = `${origin}/?query=test`;
+      const client = new undici.Client(origin);
+      const res = await client.request({
+        path: fullUrl,
+        method: 'GET',
+      });
+      await consumeResponseBody(res.body);
+
+      spans = memoryExporter.getFinishedSpans();
+      const span = spans[0];
+      assert.ok(span, 'a span is present');
+      assert.strictEqual(spans.length, 1);
+      assertSpan(span, {
+        hostname: 'localhost',
+        httpStatusCode: res.statusCode,
+        httpMethod: 'GET',
+        path: '/',
+        query: '?query=test',
+        resHeaders: res.headers,
+      });
+      assert.strictEqual(span.attributes['url.full'], fullUrl);
+    });
   });
 });