PR feedback

Author: JamesNK

src/Http/Http.Extensions/src/HttpRequestJsonExtensions.cs

@@ -83,10 +83,10 @@ public static ValueTask<TValue> ReadFromJsonAsync<TValue>(
             return JsonSerializer.DeserializeAsync(request.Body, type, options, cancellationToken);
         }
 
-        private static JsonSerializerOptions ResolveSerializerOptions(HttpContext? httpContext)
+        private static JsonSerializerOptions ResolveSerializerOptions(HttpContext httpContext)
         {
             // Attempt to resolve options from DI then fallback to default options
-            return httpContext?.RequestServices?.GetService<IOptions<JsonOptions>>()?.Value?.SerializerOptions ?? JsonOptions.DefaultSerializerOptions;
+            return httpContext.RequestServices?.GetService<IOptions<JsonOptions>>()?.Value?.SerializerOptions ?? JsonOptions.DefaultSerializerOptions;
         }
 
         private static InvalidOperationException CreateContentTypeError(HttpRequest request)

src/Http/Http.Extensions/src/HttpResponseJsonExtensions.cs

@@ -105,10 +105,10 @@ public static Task WriteAsJsonAsync(
             return JsonSerializer.SerializeAsync(response.Body, value, type, options, cancellationToken);
         }
 
-        private static JsonSerializerOptions ResolveSerializerOptions(HttpContext? httpContext)
+        private static JsonSerializerOptions ResolveSerializerOptions(HttpContext httpContext)
         {
             // Attempt to resolve options from DI then fallback to default options
-            return httpContext?.RequestServices?.GetService<IOptions<JsonOptions>>()?.Value?.SerializerOptions ?? JsonOptions.DefaultSerializerOptions;
+            return httpContext.RequestServices?.GetService<IOptions<JsonOptions>>()?.Value?.SerializerOptions ?? JsonOptions.DefaultSerializerOptions;
         }
     }
 }