rename processor

Author: nsandstrom

src/index.ts

@@ -14,7 +14,7 @@ import {
   userAgentOnRequest,
   apiKeyNameOnRequest,
   unpackBunyanLog,
-  skipStaticRequests,
+  skipResourceRequests,
   skipMonitorRequests,
 } from './telemetryProcessors'
 
@@ -36,7 +36,7 @@ const init = (options: appinsightOptions) => {
   appInsights.defaultClient.addTelemetryProcessor(userAgentOnRequest)
   appInsights.defaultClient.addTelemetryProcessor(apiKeyNameOnRequest)
   appInsights.defaultClient.addTelemetryProcessor(unpackBunyanLog)
-  appInsights.defaultClient.addTelemetryProcessor(skipStaticRequests)
+  appInsights.defaultClient.addTelemetryProcessor(skipResourceRequests)
   appInsights.defaultClient.addTelemetryProcessor(skipMonitorRequests)
 }
 

src/telemetryProcessors.test.ts

@@ -4,7 +4,7 @@ import {
   userAgentOnRequest,
   apiKeyNameOnRequest,
   unpackBunyanLog,
-  skipStaticRequests,
+  skipResourceRequests,
   skipMonitorRequests,
 } from './telemetryProcessors'
 
@@ -199,7 +199,7 @@ describe('Telemetry procesors', () => {
           },
         },
       } as unknown as Contracts.EnvelopeTelemetry
-      const callResult = skipStaticRequests(envelope)
+      const callResult = skipResourceRequests(envelope)
       expect(callResult).toBe(false)
     })
     it('does not log GET request when url has /assets/', () => {
@@ -212,7 +212,7 @@ describe('Telemetry procesors', () => {
           },
         },
       } as unknown as Contracts.EnvelopeTelemetry
-      const callResult = skipStaticRequests(envelope)
+      const callResult = skipResourceRequests(envelope)
       expect(callResult).toBe(false)
     })
     it('logs request that happens to have the word "static" in the url', () => {
@@ -225,7 +225,7 @@ describe('Telemetry procesors', () => {
           },
         },
       } as unknown as Contracts.EnvelopeTelemetry
-      const callResult = skipStaticRequests(envelope)
+      const callResult = skipResourceRequests(envelope)
       expect(callResult).toBe(true)
     })
     it('logs request that happens to have the word "assets" in the url', () => {
@@ -238,7 +238,7 @@ describe('Telemetry procesors', () => {
           },
         },
       } as unknown as Contracts.EnvelopeTelemetry
-      const callResult = skipStaticRequests(envelope)
+      const callResult = skipResourceRequests(envelope)
       expect(callResult).toBe(true)
     })
     it('logs requests where url ends with "static/', () => {
@@ -251,7 +251,7 @@ describe('Telemetry procesors', () => {
           },
         },
       } as unknown as Contracts.EnvelopeTelemetry
-      const callResult = skipStaticRequests(envelope)
+      const callResult = skipResourceRequests(envelope)
       expect(callResult).toBe(true)
     })
     it('logs requests where url ends with "assets/', () => {
@@ -264,7 +264,7 @@ describe('Telemetry procesors', () => {
           },
         },
       } as unknown as Contracts.EnvelopeTelemetry
-      const callResult = skipStaticRequests(envelope)
+      const callResult = skipResourceRequests(envelope)
       expect(callResult).toBe(true)
     })
     it('logs a non-GET request when url has /static/', () => {
@@ -277,7 +277,7 @@ describe('Telemetry procesors', () => {
           },
         },
       } as unknown as Contracts.EnvelopeTelemetry
-      const callResult = skipStaticRequests(envelope)
+      const callResult = skipResourceRequests(envelope)
       expect(callResult).toBe(true)
     })
     it('logs a non-GET request when url has /assets/', () => {
@@ -290,7 +290,7 @@ describe('Telemetry procesors', () => {
           },
         },
       } as unknown as Contracts.EnvelopeTelemetry
-      const callResult = skipStaticRequests(envelope)
+      const callResult = skipResourceRequests(envelope)
       expect(callResult).toBe(true)
     })
     it('logs if processor fails', () => {
@@ -301,7 +301,7 @@ describe('Telemetry procesors', () => {
           baseData: { name, url: 'my-server/endpoint/static/media' },
         },
       } as unknown as Contracts.EnvelopeTelemetry
-      const callResult = skipStaticRequests(envelope)
+      const callResult = skipResourceRequests(envelope)
       expect(callResult).toBe(true)
     })
   })

src/telemetryProcessors.ts

@@ -62,7 +62,7 @@ export const unpackBunyanLog = (envelope: appInsights.Contracts.EnvelopeTelemetr
 // Ignore logging any requests to static resources and assets
 // If url matches /<something>/static/<something>
 // If url matches /<something>/assets/<something>
-export const skipStaticRequests = (envelope: appInsights.Contracts.EnvelopeTelemetry) => {
+export const skipResourceRequests = (envelope: appInsights.Contracts.EnvelopeTelemetry) => {
   try {
     if (envelope.data?.baseType !== 'RequestData') return true
     if (!envelope.data.baseData?.url) return true