fix(otlp-exporter-base): use dynamic import instead of require in esm…

…/esnext builds
open-telemetry · Nov 29, 2024 · dbc8c5b · dbc8c5b
1 parent fc94e46
commit dbc8c5b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/experimental/packages/otlp-exporter-base/src/transport/http-exporter-transport.ts b/experimental/packages/otlp-exporter-base/src/transport/http-exporter-transport.ts
@@ -35,11 +35,9 @@ class HttpExporterTransport implements IExporterTransport {
   async send(data: Uint8Array, timeoutMillis: number): Promise<ExportResponse> {
     if (this._send == null) {
       // Lazy require to ensure that http/https is not required before instrumentations can wrap it.
-      const {
-        sendWithHttp,
-        createHttpAgent,
-        // eslint-disable-next-line @typescript-eslint/no-var-requires
-      } = require('./http-transport-utils');
+      const { sendWithHttp, createHttpAgent } = await import(
+        './http-transport-utils'
+      );
       this._agent = createHttpAgent(
         this._parameters.url,
         this._parameters.agentOptions

diff --git a/experimental/packages/otlp-exporter-base/tsconfig.esm.json b/experimental/packages/otlp-exporter-base/tsconfig.esm.json
@@ -3,7 +3,8 @@
   "compilerOptions": {
     "outDir": "build/esm",
     "rootDir": "src",
-    "tsBuildInfoFile": "build/esm/tsconfig.esm.tsbuildinfo"
+    "tsBuildInfoFile": "build/esm/tsconfig.esm.tsbuildinfo",
+    "module": "es2020"
   },
   "include": [
     "src/**/*.ts"