fix(otlp-exporter-base): remove brackets from IPv6 hostname in HTTP transport (#6632)

Signed-off-by: Raphael Torquato <raphael.martini@live.com>

Author: raphaeltorquat0

experimental/CHANGELOG.md

@@ -52,6 +52,7 @@ For notes on migrating to 2.x / 0.200.x see [the upgrade guide](doc/upgrade-to-2
 
 ### :bug: Bug Fixes
 
+* fix(otlp-exporter-base): fix IPv6 address handling by passing URL object to http.request [#5768](https://github.com/open-telemetry/opentelemetry-js/issues/5768) @raphaeltorquat0
 * fix(instrumentation-xml-http-request): avoid unwrapping `XMLHttpRequest` API when disabling [#6611](https://github.com/open-telemetry/opentelemetry-js/pull/6611) @david-luna
 * fix(instrumentation-fetch): tolerate non-writable `globalThis.fetch` and fix premature `_isEnabled` / `_isFetchPatched` flips in `enable()` @brunorodmoreira
 * fix(instrumentation-xhr): resolve relative URLs before matching `ignoreUrls` [#6551](https://github.com/open-telemetry/opentelemetry-js/pull/6551) @Maximiliano-Zeballos

experimental/packages/otlp-exporter-base/src/transport/http-transport-utils.ts

@@ -54,15 +54,12 @@ export function sendWithHttp(
     }
 
     const options: http.RequestOptions | https.RequestOptions = {
-      hostname: parsedUrl.hostname,
-      port: parsedUrl.port,
-      path: parsedUrl.pathname,
       method: 'POST',
       headers,
       agent,
     };
 
-    const req = request(options, (res: http.IncomingMessage) => {
+    const req = request(parsedUrl, options, (res: http.IncomingMessage) => {
       const responseData: Buffer[] = [];
       let responseSize = 0;
       res.on('data', (chunk: Buffer) => {

experimental/packages/otlp-exporter-base/test/node/http-exporter-transport.test.ts

@@ -106,6 +106,47 @@ describe('HttpExporterTransport', function () {
       );
     });
 
+    it('returns success when sending to IPv6 localhost address', async function () {
+      // arrange
+      const expectedResponseData = Buffer.from([4, 5, 6]);
+      server = http.createServer((_, res) => {
+        res.statusCode = 200;
+        res.write(expectedResponseData);
+        res.end();
+      });
+
+      // Listen on IPv6 localhost - skip test if IPv6 is not available
+      try {
+        await new Promise<void>((resolve, reject) => {
+          server!.listen(0, '::1', () => resolve());
+          server!.once('error', reject);
+        });
+      } catch (err: unknown) {
+        if ((err as NodeJS.ErrnoException).code === 'EADDRNOTAVAIL') {
+          this.skip();
+        }
+        throw err;
+      }
+      const port = (server!.address() as any).port;
+
+      const transport = createHttpExporterTransport({
+        url: `http://[::1]:${port}`,
+        headers: async () => ({}),
+        compression: 'none',
+        agentFactory: () => new http.Agent(),
+      });
+
+      // act
+      const result = await transport.send(sampleRequestData, 1000);
+
+      // assert
+      assert.strictEqual(result.status, 'success');
+      assert.deepEqual(
+        (result as ExportResponseSuccess).data,
+        expectedResponseData
+      );
+    });
+
     it('returns retryable on retryable status', async function () {
       //arrange
       server = http.createServer((_, res) => {

experimental/packages/otlp-exporter-base/test/node/http-transport-utils.test.ts

@@ -26,14 +26,17 @@ describe('compressAndSend', function () {
 });
 
 describe('sendWithHttp', function () {
-  const requestFn: typeof http.request = (
-    opts: any,
-    cb: any
+  let sentUserAgent: string;
+
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  const requestFn: any = (
+    url: URL,
+    opts: http.RequestOptions,
+    cb?: (res: http.IncomingMessage) => void
   ): http.ClientRequest => {
-    sentUserAgent = opts.headers['User-Agent'];
-    return http.request(opts, cb).destroy();
+    sentUserAgent = (opts.headers as Record<string, string>)['User-Agent'];
+    return http.request(url, opts, cb).destroy();
   };
-  let sentUserAgent: string;
 
   beforeEach(function () {
     sentUserAgent = '';