Don't set form-urlencoded header for Get requests

nickelc · nickelc · commit 628c7bf54058 · 2022-10-23T14:53:34.000+02:00
The C# runtime used by Unity fails for Get requests with
`System.Net.ProtocolViolationException: Cannot send a content-body with this verb-type.`
diff --git a/Modio/Http/Connection.cs b/Modio/Http/Connection.cs
@@ -70,7 +70,7 @@ private HttpRequestMessage BuildRequest(Request request)
                 {
                     req.Content = content;
                 }
-                else
+                else if (request.Method != HttpMethod.Get)
                 {
                     // It's safe to always set the content type to `x-www-form-urlencoded` if no content is set.
                     req.Content = new FormUrlEncodedContent(Enumerable.Empty<KeyValuePair<string?, string?>>());

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ private HttpRequestMessage BuildRequest(Request request)`
`70`	`70`	`{`
`71`	`71`	`req.Content = content;`
`72`	`72`	`}`
`73`		`- else`
	`73`	`+ else if (request.Method != HttpMethod.Get)`
`74`	`74`	`{`
`75`	`75`	// It's safe to always set the content type to `x-www-form-urlencoded` if no content is set.
`76`	`76`	`req.Content = new FormUrlEncodedContent(Enumerable.Empty<KeyValuePair<string?, string?>>());`