add logic to record performance for all API calls

samholton · Jun 2, 2022 · 319161b · 319161b
1 parent 75dee61
commit 319161b
Showing 1 changed file with 26 additions and 2 deletions.
diff --git a/src/plugins/event-plugins/ResourcePlugin.ts b/src/plugins/event-plugins/ResourcePlugin.ts
@@ -76,7 +76,20 @@ export class ResourcePlugin extends InternalPlugin {
                     // Out of n resource events, x events are recorded using Observer API
                     const type: ResourceType = getResourceFileType(event.name);
                     if (this.config.recordAllTypes.includes(type)) {
-                        recordAll.push(event);
+                        // record ALL fetch/xmlhttprequest requests ignoring this.config.eventLimit
+                        // requires ResourceType.OTHER to be added to this.config.recordAllTypes
+                        // added because observer was capturing events but record event blocks only ran on page load
+                        // probably a better way to do this, but quick fix for our use case to capture API calls
+                        const prt: PerformanceResourceTiming = event as PerformanceResourceTiming;
+                        if (
+                            ['fetch', 'xmlhttprequest'].includes(
+                                prt.initiatorType
+                            )
+                        ) {
+                            this.recordResourceEvent(prt);
+                        } else {
+                            recordAll.push(event);
+                        }
                     } else if (this.config.sampleTypes.includes(type)) {
                         sample.push(event);
                     }
@@ -93,7 +106,18 @@ export class ResourcePlugin extends InternalPlugin {
             events.forEach((event) => {
                 const type: ResourceType = getResourceFileType(event.name);
                 if (this.config.recordAllTypes.includes(type)) {
-                    recordAll.push(event);
+                    // record ALL fetch/xmlhttprequest requests ignoring this.config.eventLimit
+                    // requires ResourceType.OTHER to be added to this.config.recordAllTypes
+                    // added because observer was capturing events but record event blocks only ran on page load
+                    // probably a better way to do this, but quick fix for our use case to capture API calls
+                    const prt: PerformanceResourceTiming = event as PerformanceResourceTiming;
+                    if (
+                        ['fetch', 'xmlhttprequest'].includes(prt.initiatorType)
+                    ) {
+                        this.recordResourceEvent(prt);
+                    } else {
+                        recordAll.push(event);
+                    }
                 } else if (this.config.sampleTypes.includes(type)) {
                     sample.push(event);
                 }