Merge pull request #762 from bart-degreed/fix-kestrel-error-on-delete

Fixed: exception logged in Kestrel on Delete request

Author: Bart Koelman

src/JsonApiDotNetCore/Formatters/JsonApiWriter.cs

@@ -77,9 +77,19 @@ private string SerializeResponse(object contextObject, HttpStatusCode statusCode
                 throw new UnsuccessfulActionResultException(problemDetails);
             }
 
-            if (contextObject == null && !IsSuccessStatusCode(statusCode))
+            if (contextObject == null)
             {
-                throw new UnsuccessfulActionResultException(statusCode);
+                if (!IsSuccessStatusCode(statusCode))
+                {
+                    throw new UnsuccessfulActionResultException(statusCode);
+                }
+
+                if (statusCode == HttpStatusCode.NoContent || statusCode == HttpStatusCode.ResetContent ||
+                    statusCode == HttpStatusCode.NotModified)
+                {
+                    // Prevent exception from Kestrel server, caused by writing data:null json response.
+                    return null;
+                }
             }
 
             contextObject = WrapErrors(contextObject);

test/NoEntityFrameworkTests/WorkItemTests.cs

@@ -146,7 +146,7 @@ await ExecuteOnDbContextAsync(async dbContext =>
             string responseBody = await response.Content.ReadAsStringAsync();
             var document = JsonConvert.DeserializeObject<Document>(responseBody);
 
-            Assert.Null(document.Data);
+            Assert.Null(document);
         }
 
         private async Task ExecuteOnDbContextAsync(Func<AppDbContext, Task> asyncAction)