{"push_id":740335114,"size":1,"distinct_size":1,"ref":"refs/heads/mono-4.2.0-pre2-branch","head":"c109f9ca03d38e608cbd85cb2fdf8cfaf55bb97c","before":"3b17abfb9ba9eeb7be6e1fd033ce3118060cdaaa","commits":[{"sha":"c109f9ca03d38e608cbd85cb2fdf8cfaf55bb97c","author":{"email":"cfee4254dc96b3ae9011318adda39850f606bda0@xamarin.com","name":"Martin Baulig"},"message":"Revert \"[System] Don't assume WebConnectionStream of unknown http methods require write stream. Fixes #31830\"\n\nThis reverts commit 8ccfe0f8ff71d18151f8aad90d0855dfd2d2a723.\n\nThis commit breaks the Content-Type logic for existing HTTP verbs such as for instance\nusing DELETE with an empty body [1].  Added two new test cases [2] and [3] to the stable\nbranch of the web-tests (https://github.com/xamarin/web-tests/tree/stable) for the particular\nscenario from bug #31830 as well as the default HttpClient.SendAsync() without a Content [4].\n\nThe rules are as follows:\n\n1.) HttpWebRequest adds the Content-Length header if and only if you're calling\n    GetRequestStream(); a length of 0 will be used if you don't actually write\n    any content.\n\n    This is true for POST, PUT, DELETE and \"obscute\" verbs.\n\n2.) HttpClient.SendAsync() always adds a Content-Length header.\n\nTests:\n\n[1] https://github.com/xamarin/web-tests/blob/31b8fed0ec1283b8c0693bb14d1ac266792c1020/Xamarin.WebTests/Xamarin.WebTests.Tests/TestPost.cs#L84\n\n[2] https://github.com/xamarin/web-tests/blob/31b8fed0ec1283b8c0693bb14d1ac266792c1020/Xamarin.WebTests/Xamarin.WebTests.Tests/TestPost.cs#L296\n\n[3] https://github.com/xamarin/web-tests/blob/31b8fed0ec1283b8c0693bb14d1ac266792c1020/Xamarin.WebTests/Xamarin.WebTests.Tests/TestHttpClient.cs#L134\n\n[4] https://github.com/xamarin/web-tests/blob/31b8fed0ec1283b8c0693bb14d1ac266792c1020/Xamarin.WebTests/Xamarin.WebTests.Tests/TestHttpClient.cs#L127\n\n(cherry picked from commit 3f1ae72f1b385c3c5e4b5ac2b93d13418c020f02)","distinct":true,"url":"https://api.github.com/repos/mono/mono/commits/c109f9ca03d38e608cbd85cb2fdf8cfaf55bb97c"}]}