feat(rum-core): allow sending transactions with no spans using config (#1665)

Author: david-luna

docs/reference/configuration.md

@@ -368,3 +368,9 @@ If APM Server is deployed in an origin different than the page’s origin, you w
 ::::
 
 
+### `reportTransactionsWithoutSpans` [report-transactions-without-spans]
+
+* **Type:** Boolean
+* **Default:** `false`
+
+If you are interested also for transactions that have no spans you can set this configuration option to true. Bear in mind that sampling rate still applies even if this setting is set to `true`.

packages/rum-core/src/common/config-service.js

@@ -96,7 +96,8 @@ class Config {
       context: {},
       session: false,
       apmRequest: null,
-      sendCredentials: false
+      sendCredentials: false,
+      reportTransactionsWithoutSpans: false
     }
 
     this.events = new EventHandler()

packages/rum-core/src/performance-monitoring/performance-monitoring.js

@@ -361,7 +361,10 @@ export default class PerformanceMonitoring {
         return false
       }
 
-      if (tr.sampled && tr.spans.length === 0) {
+      const recordEmpty = this._configService.get(
+        'reportTransactionsWithoutSpans'
+      )
+      if (tr.sampled && tr.spans.length === 0 && !recordEmpty) {
         if (__DEV__) {
           this._logginService.debug(
             `transaction(${tr.id}, ${tr.name}) was discarded! Transaction does not have any spans`

packages/rum-core/test/performance-monitoring/performance-monitoring.spec.js

@@ -127,6 +127,23 @@ describe('PerformanceMonitoring', function () {
     )
   })
 
+  it('should allow transactions with no spans if enabled by configuration', () => {
+    spyOn(logger, 'debug').and.callThrough()
+    configService.setConfig({
+      ...agentConfig,
+      reportTransactionsWithoutSpans: true
+    })
+    const transaction1 = new Transaction('transaction', 'custom', {
+      id: 1,
+      startTime: 0,
+      managed: true
+    })
+    transaction1.end(600)
+    expect(transaction1.duration()).toBe(600)
+    expect(performanceMonitoring.filterTransaction(transaction1)).toBe(true)
+    expect(logger.debug).not.toHaveBeenCalled()
+  })
+
   it('should initialize and add transaction to the queue', async () => {
     performanceMonitoring.init()
     spyOn(apmServer, 'addTransaction')