update

Author: zhiyuanliang-ms

.gitignore

@@ -413,3 +413,5 @@ examples/**/**/package-lock.json
 
 # playwright test result
 test-results
+
+**/public

src/feature-management-applicationinsights-browser/src/telemetry.ts

@@ -48,9 +48,11 @@ export function trackEvent(client: ApplicationInsights, targetingId: string, eve
 export function createTargetingTelemetryInitializer(targetingContextAccessor: ITargetingContextAccessor): (item: ITelemetryItem) => void {
     return (item: ITelemetryItem) => {
         const targetingContext = targetingContextAccessor.getTargetingContext();
-        if (targetingContext?.userId === undefined) {
-            console.warn("Targeting id is undefined.");
+        if (targetingContext !== undefined) {
+            if (targetingContext?.userId === undefined) {
+                console.warn("Targeting id is undefined.");
+            }
+            item.data = {...item.data, [TARGETING_ID]: targetingContext?.userId || ""};
         }
-        item.data = {...item.data, [TARGETING_ID]: targetingContext?.userId || ""};
     };
 }

src/feature-management-applicationinsights-node/src/telemetry.ts

@@ -48,11 +48,13 @@ export function trackEvent(client: TelemetryClient, targetingId: string, event:
 export function createTargetingTelemetryProcessor(targetingContextAccessor: ITargetingContextAccessor): (envelope: Contracts.EnvelopeTelemetry) => boolean {
     return (envelope: Contracts.EnvelopeTelemetry) => {
         const targetingContext = targetingContextAccessor.getTargetingContext();
-        if (targetingContext?.userId === undefined) {
-            console.warn("Targeting id is undefined.");
+        if (targetingContext !== undefined) {
+            if (targetingContext?.userId === undefined) {
+                console.warn("Targeting id is undefined.");
+            }
+            envelope.data.baseData = envelope.data.baseData || {};
+            envelope.data.baseData.properties = {...envelope.data.baseData.properties, [TARGETING_ID]: targetingContext?.userId || ""};
         }
-        envelope.data.baseData = envelope.data.baseData || {};
-        envelope.data.baseData.properties = {...envelope.data.baseData.properties, [TARGETING_ID]: targetingContext?.userId || ""};
-        return true; // If a telemetry processor returns false, that telemetry item isn't sent.
+        return true;
     };
 }