feat: Added transaction ID to ErrorTrace event

lib/errors/index.js

@@ -67,7 +67,8 @@ class Exception {
  *                                                 [1] -> name extracted from error info,
  *                                                 [2] -> extracted error message,
  *                                                 [3] -> extracted error type,
- *                                                 [4] -> attributes
+ *                                                 [4] -> attributes,
+ *                                                 [5] -> transaction id
  */
 function createError(transaction, exception, config) {
   const error = exception.error
@@ -111,6 +112,9 @@ function createError(transaction, exception, config) {
 
   maybeAddAgentAttributes(params, exception)
 
+  if (transaction?.id) {
+    return [0, name, message, type, params, transaction.id]
+  }
   return [0, name, message, type, params]
 }
 

test/unit/api/api-set-user-id.test.js

@@ -63,7 +63,7 @@ tap.test('Agent API = set user id', (t) => {
       api.setUserID(id)
       const exception = new Exception(new Error('Test error.'))
       const [...data] = createError(tx, exception, agent.config)
-      const params = data.pop()
+      const params = data[data.length - 2]
       t.equal(params.agentAttributes['enduser.id'], id, 'should set enduser.id attribute on error')
       t.end()
     })

test/unit/errors/error-collector.test.js

@@ -199,6 +199,82 @@ tap.test('Errors', (t) => {
     })
   })
 
+  t.test('transaction id with distributed tracing enabled', (t) => {
+    t.autoend()
+    let errorJSON
+    let transaction
+    let error
+
+    t.beforeEach(() => {
+      agent.config.distributed_tracing.enabled = true
+      error = new Error('this is an error')
+    })
+
+    t.test('should have a transaction id when there is a transaction', (t) => {
+      transaction = new Transaction(agent)
+
+      agent.errors.add(transaction, error)
+      agent.errors.onTransactionFinished(transaction)
+
+      const errorTraces = getErrorTraces(agent.errors)
+      errorJSON = errorTraces[0]
+
+      const transactionId = errorJSON[5]
+      t.not(transactionId, undefined)
+      transaction.end()
+      t.end()
+    })
+
+    t.test('should not have a transaction id when there is no transaction', (t) => {
+      agent.errors.add(null, error)
+
+      const errorTraces = getErrorTraces(agent.errors)
+      errorJSON = errorTraces[0]
+
+      const transactionId = errorJSON[5]
+      t.notOk(transactionId)
+      t.end()
+    })
+  })
+
+  t.test('transaction id with distributed tracing disabled', (t) => {
+    t.autoend()
+    let errorJSON
+    let transaction
+    let error
+
+    t.beforeEach(() => {
+      agent.config.distributed_tracing.enabled = false
+      error = new Error('this is an error')
+    })
+
+    t.test('should have a transaction id when there is a transaction', (t) => {
+      transaction = new Transaction(agent)
+
+      agent.errors.add(transaction, error)
+      agent.errors.onTransactionFinished(transaction)
+
+      const errorTraces = getErrorTraces(agent.errors)
+      errorJSON = errorTraces[0]
+
+      const transactionId = errorJSON[5]
+      t.not(transactionId, undefined)
+      transaction.end()
+      t.end()
+    })
+
+    t.test('should not have a transaction id when there is no transaction', (t) => {
+      agent.errors.add(null, error)
+
+      const errorTraces = getErrorTraces(agent.errors)
+      errorJSON = errorTraces[0]
+
+      const transactionId = errorJSON[5]
+      t.notOk(transactionId)
+      t.end()
+    })
+  })
+
   t.test('display name', (t) => {
     t.autoend()
     const PARAMS = 4
@@ -667,6 +743,17 @@ tap.test('Errors', (t) => {
         t.notHas(params, 'stack_trace')
         t.end()
       })
+
+      t.test('should have a transaction id', (t) => {
+        const transactionId = errorJSON[5]
+        t.not(transactionId, undefined)
+        t.end()
+      })
+
+      t.test('should have 6 elements in errorJson', (t) => {
+        t.equal(errorJSON.length, 6)
+        t.end()
+      })
     })
 
     t.test('with transaction agent attrs, status code, and no error', (t) => {
@@ -721,6 +808,12 @@ tap.test('Errors', (t) => {
         t.end()
       })
 
+      t.test('should have a transaction id', (t) => {
+        const transactionId = errorJSON[5]
+        t.not(transactionId, undefined)
+        t.end()
+      })
+
       t.test('should not have a request URL', (t) => {
         t.notOk(params['request.uri'])
         t.end()
@@ -826,6 +919,12 @@ tap.test('Errors', (t) => {
         t.equal(params.stack_trace[0], 'Error: Dare to be the same!')
         t.end()
       })
+
+      t.test('should not have a transaction id', (t) => {
+        const transactionId = errorJSON[5]
+        t.notOk(transactionId)
+        t.end()
+      })
     })
 
     t.test('with a thrown TypeError and a transaction with no params', (t) => {
@@ -878,6 +977,12 @@ tap.test('Errors', (t) => {
         t.equal(params.stack_trace[0], 'TypeError: Dare to be different!')
         t.end()
       })
+
+      t.test('should have a transaction id', (t) => {
+        const transactionId = errorJSON[5]
+        t.not(transactionId, undefined)
+        t.end()
+      })
     })
 
     t.test('with a thrown `TypeError` and a transaction with agent attrs', (t) => {
@@ -935,6 +1040,12 @@ tap.test('Errors', (t) => {
         t.end()
       })
 
+      t.test('should have a transaction id', (t) => {
+        const transactionId = errorJSON[5]
+        t.not(transactionId, undefined)
+        t.end()
+      })
+
       t.test('should not have a request URL', (t) => {
         t.notOk(params['request.uri'])
         t.end()
@@ -999,6 +1110,12 @@ tap.test('Errors', (t) => {
         t.notHas(errorJSON[4], 'stack_trace')
         t.end()
       })
+
+      t.test('should have a transaction id', (t) => {
+        const transactionId = errorJSON[5]
+        t.not(transactionId, undefined)
+        t.end()
+      })
     })
 
     t.test('with a thrown string and a transaction with agent parameters', (t) => {
@@ -1056,6 +1173,12 @@ tap.test('Errors', (t) => {
         t.end()
       })
 
+      t.test('should have a transaction id', (t) => {
+        const transactionId = errorJSON[5]
+        t.not(transactionId, undefined)
+        t.end()
+      })
+
       t.test('should not have a request URL', (t) => {
         t.notOk(params['request.uri'])
         t.end()

test/versioned/connect/error-intercept.tap.js

@@ -74,7 +74,7 @@ test('intercepting errors with connect 2', function (t) {
       t.equal(errors.length, 1, 'the error got traced')
 
       const error = errors[0]
-      t.equal(error.length, 5, 'format for traced error is correct')
+      t.equal(error.length, 6, 'format for traced error is correct')
       t.equal(error[3], 'TypeError', 'got the correct class for the error')
 
       server.close()