{"push_id":641534510,"size":16,"distinct_size":2,"ref":"refs/heads/kiran/log-filters","head":"c5efae5b0ec1dccea325290b64461b1babddac96","before":"2516e328aa4686938476516f3ea6abce4bcbb79f","commits":[{"sha":"323ec2fdc7de3f86ce707d95fcdf887adcea35fe","author":{"email":"b0b86c3084d86c302f3c26917ddc4c78bcc3b79e@gmail.com","name":"Henk Mollema"},"message":"Throw exception with unknown attributeName\n\nThrow a meaningful exception if the attribute to copy does not exists\nin the attributes of the TagHelperContext rather than:\n`System.InvalidOperationException : Sequence contains no matching\nelement`.\n\nAdded test too.","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/323ec2fdc7de3f86ce707d95fcdf887adcea35fe"},{"sha":"7b7667e33822793e559fd29130e0507ac0d4129a","author":{"email":"17a1bcfb12b91d5a94bc2a853ed1356b563f4b10@Outlook.com","name":"Chris Ross"},"message":"Fix DataProtection dependency.","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/7b7667e33822793e559fd29130e0507ac0d4129a"},{"sha":"92554fa634b9feea1c3ad6688e83c598f8875aca","author":{"email":"b0b86c3084d86c302f3c26917ddc4c78bcc3b79e@gmail.com","name":"Henk Mollema"},"message":"Utilize `nameof` operator","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/92554fa634b9feea1c3ad6688e83c598f8875aca"},{"sha":"9d2b1822d9e93bf74e3f1df8096b6bb2d6d43371","author":{"email":"b973317be61f798d58d714b7b626086da516278a@microsoft.com","name":"Kirthi Krishnamraju"},"message":"Fix for #819 : Attribute Routing: Resolve Name + Token Replacement","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/9d2b1822d9e93bf74e3f1df8096b6bb2d6d43371"},{"sha":"f3679f214e85b64bc4d7387e34753f4b8e01beb8","author":{"email":"9ee1eb54eabc39e905f4364545c101eb73de15aa@gmail.com","name":"Ryan Nowak"},"message":"Fix #1712 - remove reflection from validation code in MutableObjectModelBinder\n\nThis change moves [BindingBehavior(...)] and friends into the model\nmetadata layer, and removes the reflection code from\nMutableObjectModelBinder that was looking for them previously.","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/f3679f214e85b64bc4d7387e34753f4b8e01beb8"},{"sha":"a001cd4f2af13a862275f857585bde1201bc1c21","author":{"email":"18b75621cc0eeda1aa6ad261402aec709b2e75f5@microsoft.com","name":"Hao Kung"},"message":"React to Security logging change","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/a001cd4f2af13a862275f857585bde1201bc1c21"},{"sha":"9f97d25e027c4c529b005e0afbca1db5809e6a58","author":{"email":"b973317be61f798d58d714b7b626086da516278a@microsoft.com","name":"Kirthi Krishnamraju"},"message":"Moved exception messages to resource file and updated tests to validate exception messages","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/9f97d25e027c4c529b005e0afbca1db5809e6a58"},{"sha":"e48565dcd8c33c424aaa9a6a569ac0186eb784a6","author":{"email":"b973317be61f798d58d714b7b626086da516278a@microsoft.com","name":"Kirthi Krishnamraju"},"message":"Adding errors in ModelState","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/e48565dcd8c33c424aaa9a6a569ac0186eb784a6"},{"sha":"bf712263fcf51187fe970c7e177633251b6179c3","author":{"email":"b973317be61f798d58d714b7b626086da516278a@microsoft.com","name":"Kirthi Krishnamraju"},"message":"Fix for #2280 - Cannot do an \"add\" patch to IList<Anything>\nAdded unit tests","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/bf712263fcf51187fe970c7e177633251b6179c3"},{"sha":"3042a62ba95c014179dd5604dc10fecfbd4c924a","author":{"email":"b973317be61f798d58d714b7b626086da516278a@microsoft.com","name":"Kirthi Krishnamraju"},"message":"Removed Test operation code and changed T to TModel","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/3042a62ba95c014179dd5604dc10fecfbd4c924a"},{"sha":"924d50bd8f9684b08124463cf4bc677fc5f2e5a2","author":{"email":"b973317be61f798d58d714b7b626086da516278a@microsoft.com","name":"Kirthi Krishnamraju"},"message":"Added functionaltests for JsonPatchDocument","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/924d50bd8f9684b08124463cf4bc677fc5f2e5a2"},{"sha":"53ef8258bb0f7457ce17ea652f92201f1c60bed2","author":{"email":"97a0d48697f3fbe01f495fac69c561208c804a32@microsoft.com","name":"Harsh Gupta"},"message":"The model state keys for body bound models which are bound at property will use the entire model name with this change for example\nConsider\n\npublic class Person\n{\n    [FromBody]\n    public Address Address { get; set; }\n}\n\npublic class Address\n{\n   [Required]\n   public string Street { get; set; }\n\n   public int Zip { get; set; }\n}\n\nRequest body { \"Zip\" : 12345 }\nIn this case the error key would be \"prefix.Address.Street\" (assuming there is a prefix because of additional metadata/positioning for/of the Person model).\n\npublic class Person\n{\n       [Required]\n       public string Name { get; set; }\n}\n\npublic void Action([FromBody]Person p)\n{\n}\nRequest body { }\nIn this case the prefix gets ignored and the error key is Name.\nPlease note this is so that we are compatible with MVC 5.0\n\npublic class Person\n{\n       [Required]\n       public string Name { get; set; }\n}\n\npublic void Action([FromBody][ModelBinder(Name = \"prefix\")] Person p)\n{\n}\n\npublic void Action2([FromBody][Bind(Name = \"prefix\")] Person p)\n{\n}\nRequest body { }\nIn both these cases (Action and Action2) the prefix gets ignored and the error key is Name.\nThis is a slight improvement from mvc, as in MVC the action parameter would be null.\n\nThe followup for this would be to fix #2416 -\nThis PR ignores the validation assuming that #2416 will address the issues and update the test.\n\nNOTE: previous versions of mvc did not have property binding and hence there is no precedence in this case. For MVC and Web API it was possible to body bind an action parameter which used an empty prefix instead of a parameter name for adding errors to model state (In case of MVC if a custom prefix was provided, it failed binding from body i.e the parameter was null).","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/53ef8258bb0f7457ce17ea652f92201f1c60bed2"},{"sha":"86fcacea92839a2d260e447f4427bf3048e06be9","author":{"email":"97a0d48697f3fbe01f495fac69c561208c804a32@microsoft.com","name":"Harsh Gupta"},"message":"Fix for #2357 : We prevent assigining null values to non nullable controller properties.","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/86fcacea92839a2d260e447f4427bf3048e06be9"},{"sha":"4df7d52210555f3fe2264241a5d53ed5e5e3cf06","author":{"email":"97a0d48697f3fbe01f495fac69c561208c804a32@microsoft.com","name":"Harsh Gupta"},"message":"PR Comments","distinct":false,"url":"https://api.github.com/repos/aspnet/Mvc/commits/4df7d52210555f3fe2264241a5d53ed5e5e3cf06"},{"sha":"1132d65b8dd8aa73689398eaeb5172d9073124b0","author":{"email":"21b71ee811f8d93766ad9641ace4179d8cdcb108@microsoft.com","name":"Kiran Challa"},"message":"Logging for filters","distinct":true,"url":"https://api.github.com/repos/aspnet/Mvc/commits/1132d65b8dd8aa73689398eaeb5172d9073124b0"},{"sha":"c5efae5b0ec1dccea325290b64461b1babddac96","author":{"email":"21b71ee811f8d93766ad9641ace4179d8cdcb108@microsoft.com","name":"Kiran Challa"},"message":"Updated","distinct":true,"url":"https://api.github.com/repos/aspnet/Mvc/commits/c5efae5b0ec1dccea325290b64461b1babddac96"}]}