Remove HttpResponseMessage allocation in hot path

bkoelman · bkoelman · commit 655c2da6f8fb · 2026-03-08T16:39:19.000+01:00
diff --git a/src/JsonApiDotNetCore/Serialization/Response/JsonApiWriter.cs b/src/JsonApiDotNetCore/Serialization/Response/JsonApiWriter.cs
@@ -118,7 +118,7 @@ private static bool CanWriteBody(HttpStatusCode statusCode)
 
     private static bool IsSuccessStatusCode(HttpStatusCode statusCode)
     {
-        return new HttpResponseMessage(statusCode).IsSuccessStatusCode;
+        return (int)statusCode is >= 200 and <= 299;
     }
 
     private string RenderModel(object? model)

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ private static bool CanWriteBody(HttpStatusCode statusCode)`
`118`	`118`
`119`	`119`	`private static bool IsSuccessStatusCode(HttpStatusCode statusCode)`
`120`	`120`	`{`
`121`		`- return new HttpResponseMessage(statusCode).IsSuccessStatusCode;`
	`121`	`+ return (int)statusCode is >= 200 and <= 299;`
`122`	`122`	`}`
`123`	`123`
`124`	`124`	`private string RenderModel(object? model)`