Remove S6964

Author: neoscie

CHANGELOG.md

@@ -35,7 +35,6 @@ and adheres to a project-specific [Versioning](/README.md).
 - S6960: Controllers should not have mixed responsibilities
 - S6961: API Controllers should derive from ControllerBase instead of Controller
 - S6962: You should pool HTTP connections with HttpClientFactory
-- S6964: Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting.
 - S6965: REST API actions should be annotated with an HTTP verb attribute
 - S6966: Awaitable method should be used
 - S6967: ModelState.IsValid should be called in controller actions

build/Neolution.CodeAnalysis.globalconfig

@@ -1,4 +1,4 @@
-# Top level entry required to mark this as a global AnalyzerConfig file
+# Top level entry required to mark this as a global AnalyzerConfig file
 # NOTE: Do not create section headers for configuration entries
 is_global = true
 
@@ -274,6 +274,10 @@ dotnet_diagnostic.S6608.severity = none
 ; This rule has no effect for our projects, since we mainly use Endpoints which limits developers to only implement one action per "Controller".
 dotnet_diagnostic.S6931.severity = none
 
+# S6964: Value type property used as input in a controller action should be nullable, required or annotated with the JsonRequiredAttribute to avoid under-posting
+; This is handled by a custom model binder in our project templates that produces a bad request response in case of under-posting 
+dotnet_diagnostic.S6964.severity = none
+
 # S107: Methods should not have too many parameters
 dotnet_diagnostic.S107.severity = warning