Allow extra properties iff they are null

microsoft · Sep 1, 2022 · db6c955 · db6c955
1 parent 9c810d0
commit db6c955
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/ApiService/ApiService/onefuzzlib/Request.cs b/src/ApiService/ApiService/onefuzzlib/Request.cs
@@ -42,8 +42,12 @@ public static async Async.Task<OneFuzzResult<T>> ParseRequest<T>(HttpRequestData
                 // properties set:
                 if (t.ExtensionData != null) {
                     var errors = new List<string>();
-                    foreach (var property in t.ExtensionData.Keys) {
-                        errors.Add($"Unexpected property: \"{property}\"");
+                    foreach (var (name, value) in t.ExtensionData) {
+                        // allow additional properties if they are null,
+                        // otherwise produce an error
+                        if (value.ValueKind != JsonValueKind.Null) {
+                            errors.Add($"Unexpected property: \"{name}\"");
+                        }
                     }
 
                     return new Error(