add null/undefined check for filter parameters

Author: linglingye001

src/appConfigurationImpl.ts

@@ -900,6 +900,10 @@ export class AzureAppConfigurationImpl implements AzureAppConfiguration {
                     }
                     i++;
                 }
+
+                if (i < pageWatchers.length) {
+                    return true;
+                }
             }
             return false;
         };

src/featureManagement/featureFlagConverter.ts

@@ -94,7 +94,14 @@ export function convert(featureFlag: AzAppConfigFeatureFlag): FeatureFlag {
     return result;
 }
 
-function parseParameterValue(value: string): unknown {
+function parseParameterValue(value: unknown): unknown {
+    if (value === undefined || value === null) {
+        return value;
+    }
+    if (typeof value !== "string") {
+        return value;
+    }
+
     const trimmedValue = value.trim();
     if (trimmedValue.length > 0 && (trimmedValue[0] === "{" || trimmedValue[0] === "[")) {
         try {

test/featureFlag.test.ts

@@ -570,7 +570,9 @@ describe("enhanced feature flags", function () {
                             Audience: JSON.stringify(audience),
                             JsonArray: "  [\"one\",\"two\"]  ",
                             PlainText: "not-json",
-                            Percentage: "50"
+                            Percentage: "50",
+                            NullValue: null,
+                            UndefinedValue: undefined
                         }
                     }]
                 }
@@ -588,6 +590,8 @@ describe("enhanced feature flags", function () {
         expect(parameters.JsonArray).deep.equals(["one", "two"]);
         expect(parameters.PlainText).equals("not-json");
         expect(parameters.Percentage).equals("50");
+        expect(parameters.NullValue).equals(null);
+        expect(parameters).has.property("UndefinedValue", undefined);
     });
 
     it("should preserve invalid JSON in enhanced feature flag filter parameters as a string", () => {