fb

Author: BrennanConroy

src/Mvc/Mvc.Core/src/ModelBinding/Binders/BodyModelBinder.cs

@@ -144,10 +144,15 @@ public async Task BindModelAsync(ModelBindingContext bindingContext)
 
         if (formatter == null)
         {
-            if (AllowEmptyBody && httpContext.Features.Get<IHttpRequestBodyDetectionFeature>()?.CanHaveBody == false)
+            if (AllowEmptyBody)
             {
-                bindingContext.Result = ModelBindingResult.Success(model: null);
-                return;
+                var hasBody = httpContext.Features.Get<IHttpRequestBodyDetectionFeature>()?.CanHaveBody;
+                hasBody ??= httpContext.Request.ContentLength is not null && httpContext.Request.ContentLength == 0;
+                if (hasBody == false)
+                {
+                    bindingContext.Result = ModelBindingResult.Success(model: null);
+                    return;
+                }
             }
 
             _logger.NoInputFormatterSelected(formatterContext);

src/Mvc/Mvc.Core/test/ModelBinding/Binders/BodyModelBinderTests.cs

@@ -195,6 +195,62 @@ public async Task BindModel_PassesAllowEmptyInputOptionViaContext(bool treatEmpt
             Times.Once);
     }
 
+    [Fact]
+    public async Task BindModel_SetsModelIfAllowEmpty()
+    {
+        // Arrange
+        var mockInputFormatter = new Mock<IInputFormatter>();
+        mockInputFormatter.Setup(f => f.CanRead(It.IsAny<InputFormatterContext>()))
+            .Returns(false);
+        var inputFormatter = mockInputFormatter.Object;
+
+        var provider = new TestModelMetadataProvider();
+        provider.ForType<Person>().BindingDetails(d => d.BindingSource = BindingSource.Body);
+
+        var bindingContext = GetBindingContext(
+            typeof(Person),
+            metadataProvider: provider);
+        bindingContext.BinderModelName = "custom";
+
+        var binder = CreateBinder(new[] { inputFormatter }, treatEmptyInputAsDefaultValueOption : true);
+
+        // Act
+        await binder.BindModelAsync(bindingContext);
+
+        // Assert
+        Assert.True(bindingContext.Result.IsModelSet);
+        Assert.Null(bindingContext.Result.Model);
+        Assert.True(bindingContext.ModelState.IsValid);
+    }
+
+    [Fact]
+    public async Task BindModel_FailsIfNotAllowEmpty()
+    {
+        // Arrange
+        var mockInputFormatter = new Mock<IInputFormatter>();
+        mockInputFormatter.Setup(f => f.CanRead(It.IsAny<InputFormatterContext>()))
+            .Returns(false);
+        var inputFormatter = mockInputFormatter.Object;
+
+        var provider = new TestModelMetadataProvider();
+        provider.ForType<Person>().BindingDetails(d => d.BindingSource = BindingSource.Body);
+
+        var bindingContext = GetBindingContext(
+            typeof(Person),
+            metadataProvider: provider);
+        bindingContext.BinderModelName = "custom";
+
+        var binder = CreateBinder(new[] { inputFormatter }, treatEmptyInputAsDefaultValueOption: false);
+
+        // Act
+        await binder.BindModelAsync(bindingContext);
+
+        // Assert
+        Assert.False(bindingContext.ModelState.IsValid);
+        Assert.Single(bindingContext.ModelState[bindingContext.BinderModelName].Errors);
+        Assert.Equal("Unsupported content type ''.", bindingContext.ModelState[bindingContext.BinderModelName].Errors[0].Exception.Message);
+    }
+
     // Throwing InputFormatterException
     [Fact]
     public async Task BindModel_CustomFormatter_ThrowingInputFormatterException_AddsErrorToModelState()