Telemetry updates

Author: estruyf

CHANGELOG.JSON

@@ -1,13 +1,24 @@
 {
   "versions": [
+    {
+      "version": "1.5.`",
+      "changes": {
+        "new": [],
+        "enhancements": [],
+        "fixes": [
+          "Fix issue where Application Insights is not defined [#81](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/81)"
+        ]
+      },
+      "contributions": ["Asish Padhy", "Alex Terentiev"]
+    },
     {
       "version": "1.5.0",
       "changes": {
         "new": [
           "New `PeoplePicker` control added [#19](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/19)"
         ],
         "enhancements": [
-          "Added a properties to the `TaxonomyPicker` to specify which terms are disabled/not-selectable [#82](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/82)"
+          "Added properties to the `TaxonomyPicker` to specify which terms are disabled/not-selectable [#82](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/82)"
         ],
         "fixes": [
           "Bug in `TaxonomyPicker` where values are not updated by an async change [#83](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/83)",

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Releases
 
+## 1.5.`
+
+**Fixes**
+
+- Fix issue where Application Insights is not defined [#81](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/81)
+
 ## 1.5.0
 
 **New control(s)**
@@ -8,7 +14,7 @@
 
 **Enhancements**
 
-- Added a properties to the `TaxonomyPicker` to specify which terms are disabled/not-selectable [#82](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/82)
+- Added properties to the `TaxonomyPicker` to specify which terms are disabled/not-selectable [#82](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/82)
 
 **Fixes**
 

docs/documentation/docs/about/release-notes.md

@@ -1,5 +1,11 @@
 # Releases
 
+## 1.5.`
+
+**Fixes**
+
+- Fix issue where Application Insights is not defined [#81](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/81)
+
 ## 1.5.0
 
 **New control(s)**
@@ -8,7 +14,7 @@
 
 **Enhancements**
 
-- Added a properties to the `TaxonomyPicker` to specify which terms are disabled/not-selectable [#82](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/82)
+- Added properties to the `TaxonomyPicker` to specify which terms are disabled/not-selectable [#82](https://github.com/SharePoint/sp-dev-fx-controls-react/issues/82)
 
 **Fixes**
 

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@pnp/spfx-controls-react",
   "description": "Reusable React controls for SharePoint Framework solutions",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "engines": {
     "node": ">=0.10.0"
   },

src/common/appInsights/index.ts

@@ -4,28 +4,41 @@ import {Environment,EnvironmentType} from "@microsoft/sp-core-library";
 
 const controlType = "react";
 
-AppInsights.downloadAndSetup({ instrumentationKey: "9f59b81e-d2ed-411e-a961-8bcf3f7f04d0" });
-
-appInsights.queue.push(() => {
-  // Remove operation name from the telemetry
-  if (AppInsights.context && AppInsights.context.operation && AppInsights.context.operation.name) {
-    AppInsights.context.operation.name = null;
-  }
+if (typeof AppInsights !== "undefined") {
+  AppInsights.downloadAndSetup({
+    instrumentationKey: "9f59b81e-d2ed-411e-a961-8bcf3f7f04d0",
+    disableExceptionTracking: true,
+    disableAjaxTracking: true
+  });
+}
 
-  // Filter out telemetry data
-  appInsights.context.addTelemetryInitializer((envelope: Microsoft.ApplicationInsights.IEnvelope) => {
-    const telemetryItem = envelope.data.baseData;
-    // Only send telemetry data if it contains data of this library
-    if (!telemetryItem.properties || !telemetryItem.properties.controlType) {
-      return false;
+if (typeof appInsights !== "undefined") {
+  appInsights.queue.push(() => {
+    // Remove operation name from the telemetry
+    if (AppInsights.context && AppInsights.context.operation && AppInsights.context.operation.name) {
+      AppInsights.context.operation.name = null;
     }
 
-    // Only send telemetry if it is coming from the right control type
-    if (telemetryItem.properties && telemetryItem.properties.controlType && telemetryItem.properties.controlType !== controlType) {
-      return false;
-    }
+    // Filter out telemetry data
+    appInsights.context.addTelemetryInitializer((envelope: Microsoft.ApplicationInsights.IEnvelope) => {
+      const telemetryItem = envelope.data.baseData;
+      // Only send telemetry data if it contains data of this library
+      if (!telemetryItem.properties || !telemetryItem.properties.controlType) {
+        return false;
+      }
+
+      // Check if the type of data is only EventData, otherwise this may not be sent
+      if (envelope.data && envelope.data.baseType !== "EventData") {
+        return false;
+      }
+
+      // Only send telemetry if it is coming from the right control type
+      if (telemetryItem.properties && telemetryItem.properties.controlType && telemetryItem.properties.controlType !== controlType) {
+        return false;
+      }
+    });
   });
-});
+}
 
 export function track(componentName: string, properties: any = {}): void {
   AppInsights.trackEvent(componentName, {