Extract profiler start code, soon we'll be using it twice

DataDog · May 17, 2024 · 3bb306f · 3bb306f
1 parent 318f118
commit 3bb306f
Showing 1 changed file with 17 additions and 12 deletions.
diff --git a/packages/dd-trace/src/proxy.js b/packages/dd-trace/src/proxy.js
@@ -99,18 +99,7 @@ class Tracer extends NoopProxy {
       const ssiTelemetry = new SSITelemetry(config.profiling)
       ssiTelemetry.start()
       if (config.profiling.enabled) {
-        // do not stop tracer initialization if the profiler fails to be imported
-        try {
-          const profiler = require('./profiler')
-          this._profilerStarted = profiler.start(config)
-        } catch (e) {
-          log.error(e)
-          telemetryLog.publish({
-            message: e.message,
-            level: 'ERROR',
-            stack_trace: e.stack
-          })
-        }
+        this._profilerStarted = this._startProfiler(config)
       } else if (ssiTelemetry.enabled()) {
         require('./profiling/ssi-telemetry-mock-profiler').start(config)
       }
@@ -138,6 +127,22 @@ class Tracer extends NoopProxy {
     return this
   }
 
+  _startProfiler (config) {
+    // do not stop tracer initialization if the profiler fails to be imported
+    try {
+      return require('./profiler').start(config)
+    } catch (e) {
+      log.error(e)
+      if (telemetryLog.hasSubscribers) {
+        telemetryLog.publish({
+          message: e.message,
+          level: 'ERROR',
+          stack_trace: e.stack
+        })
+      }
+    }
+  }
+
   _enableOrDisableTracing (config) {
     if (config.tracing !== false) {
       if (config.appsec.enabled) {