don't compute service name unless necessary

Jordi Bertran de Balanda · tlhunter · commit 026023c87ca0 · 2023-05-18T12:44:54.000-07:00
diff --git a/packages/dd-trace/src/plugins/consumer.js b/packages/dd-trace/src/plugins/consumer.js
@@ -8,15 +8,12 @@ class ConsumerPlugin extends InboundPlugin {
   static get type () { return 'messaging' }
 
   startSpan (options) {
-    const spanDefaults = {
-      service: this.config.service || this.serviceName(),
-      kind: this.constructor.kind
+    if (!options.service) {
+      options.service = this.config.service || this.serviceName()
+    }
+    if (!options.kind) {
+      options.kind = this.constructor.kind
     }
-    Object.keys(spanDefaults).forEach(
-      key => {
-        if (!options[key]) options[key] = spanDefaults[key]
-      }
-    )
     return super.startSpan(this.operationName(), options)
   }
 }
diff --git a/packages/dd-trace/src/plugins/producer.js b/packages/dd-trace/src/plugins/producer.js
@@ -9,9 +9,11 @@ class ProducerPlugin extends OutboundPlugin {
 
   startSpan (options) {
     const spanDefaults = {
-      service: this.config.service || this.serviceName(),
       kind: this.constructor.kind
     }
+    if (!options.service) {
+      options.service = this.config.service || this.serviceName()
+    }
     Object.keys(spanDefaults).forEach(
       key => {
         if (!options[key]) options[key] = spanDefaults[key]
