From 4e5463dadeb7e5a64a6725bb9d0956a47e618c97 Mon Sep 17 00:00:00 2001 From: Stephen Toub Date: Wed, 18 Nov 2020 17:21:45 -0500 Subject: [PATCH] Fix nullable annotations on HttpClient.Post/Put/PatchAsync (#44880) Null content is allowed. --- .../System.Net.Http/ref/System.Net.Http.cs | 24 +++++++++++----------- .../src/System/Net/Http/HttpClient.cs | 24 +++++++++++----------- 2 files changed, 24 insertions(+), 24 deletions(-) diff --git a/src/libraries/System.Net.Http/ref/System.Net.Http.cs b/src/libraries/System.Net.Http/ref/System.Net.Http.cs index 957f8f0..35406af 100644 --- a/src/libraries/System.Net.Http/ref/System.Net.Http.cs +++ b/src/libraries/System.Net.Http/ref/System.Net.Http.cs @@ -78,18 +78,18 @@ namespace System.Net.Http public System.Threading.Tasks.Task GetStringAsync(string? requestUri, System.Threading.CancellationToken cancellationToken) { throw null; } public System.Threading.Tasks.Task GetStringAsync(System.Uri? requestUri) { throw null; } public System.Threading.Tasks.Task GetStringAsync(System.Uri? requestUri, System.Threading.CancellationToken cancellationToken) { throw null; } - public System.Threading.Tasks.Task PatchAsync(string? requestUri, System.Net.Http.HttpContent content) { throw null; } - public System.Threading.Tasks.Task PatchAsync(string? requestUri, System.Net.Http.HttpContent content, System.Threading.CancellationToken cancellationToken) { throw null; } - public System.Threading.Tasks.Task PatchAsync(System.Uri? requestUri, System.Net.Http.HttpContent content) { throw null; } - public System.Threading.Tasks.Task PatchAsync(System.Uri? requestUri, System.Net.Http.HttpContent content, System.Threading.CancellationToken cancellationToken) { throw null; } - public System.Threading.Tasks.Task PostAsync(string? requestUri, System.Net.Http.HttpContent content) { throw null; } - public System.Threading.Tasks.Task PostAsync(string? requestUri, System.Net.Http.HttpContent content, System.Threading.CancellationToken cancellationToken) { throw null; } - public System.Threading.Tasks.Task PostAsync(System.Uri? requestUri, System.Net.Http.HttpContent content) { throw null; } - public System.Threading.Tasks.Task PostAsync(System.Uri? requestUri, System.Net.Http.HttpContent content, System.Threading.CancellationToken cancellationToken) { throw null; } - public System.Threading.Tasks.Task PutAsync(string? requestUri, System.Net.Http.HttpContent content) { throw null; } - public System.Threading.Tasks.Task PutAsync(string? requestUri, System.Net.Http.HttpContent content, System.Threading.CancellationToken cancellationToken) { throw null; } - public System.Threading.Tasks.Task PutAsync(System.Uri? requestUri, System.Net.Http.HttpContent content) { throw null; } - public System.Threading.Tasks.Task PutAsync(System.Uri? requestUri, System.Net.Http.HttpContent content, System.Threading.CancellationToken cancellationToken) { throw null; } + public System.Threading.Tasks.Task PatchAsync(string? requestUri, System.Net.Http.HttpContent? content) { throw null; } + public System.Threading.Tasks.Task PatchAsync(string? requestUri, System.Net.Http.HttpContent? content, System.Threading.CancellationToken cancellationToken) { throw null; } + public System.Threading.Tasks.Task PatchAsync(System.Uri? requestUri, System.Net.Http.HttpContent? content) { throw null; } + public System.Threading.Tasks.Task PatchAsync(System.Uri? requestUri, System.Net.Http.HttpContent? content, System.Threading.CancellationToken cancellationToken) { throw null; } + public System.Threading.Tasks.Task PostAsync(string? requestUri, System.Net.Http.HttpContent? content) { throw null; } + public System.Threading.Tasks.Task PostAsync(string? requestUri, System.Net.Http.HttpContent? content, System.Threading.CancellationToken cancellationToken) { throw null; } + public System.Threading.Tasks.Task PostAsync(System.Uri? requestUri, System.Net.Http.HttpContent? content) { throw null; } + public System.Threading.Tasks.Task PostAsync(System.Uri? requestUri, System.Net.Http.HttpContent? content, System.Threading.CancellationToken cancellationToken) { throw null; } + public System.Threading.Tasks.Task PutAsync(string? requestUri, System.Net.Http.HttpContent? content) { throw null; } + public System.Threading.Tasks.Task PutAsync(string? requestUri, System.Net.Http.HttpContent? content, System.Threading.CancellationToken cancellationToken) { throw null; } + public System.Threading.Tasks.Task PutAsync(System.Uri? requestUri, System.Net.Http.HttpContent? content) { throw null; } + public System.Threading.Tasks.Task PutAsync(System.Uri? requestUri, System.Net.Http.HttpContent? content, System.Threading.CancellationToken cancellationToken) { throw null; } [System.Runtime.Versioning.UnsupportedOSPlatformAttribute("browser")] public System.Net.Http.HttpResponseMessage Send(System.Net.Http.HttpRequestMessage request) { throw null; } [System.Runtime.Versioning.UnsupportedOSPlatformAttribute("browser")] diff --git a/src/libraries/System.Net.Http/src/System/Net/Http/HttpClient.cs b/src/libraries/System.Net.Http/src/System/Net/Http/HttpClient.cs index 0a9aa07..807bab6 100644 --- a/src/libraries/System.Net.Http/src/System/Net/Http/HttpClient.cs +++ b/src/libraries/System.Net.Http/src/System/Net/Http/HttpClient.cs @@ -376,48 +376,48 @@ namespace System.Net.Http public Task GetAsync(Uri? requestUri, HttpCompletionOption completionOption, CancellationToken cancellationToken) => SendAsync(CreateRequestMessage(HttpMethod.Get, requestUri), completionOption, cancellationToken); - public Task PostAsync(string? requestUri, HttpContent content) => + public Task PostAsync(string? requestUri, HttpContent? content) => PostAsync(CreateUri(requestUri), content); - public Task PostAsync(Uri? requestUri, HttpContent content) => + public Task PostAsync(Uri? requestUri, HttpContent? content) => PostAsync(requestUri, content, CancellationToken.None); - public Task PostAsync(string? requestUri, HttpContent content, CancellationToken cancellationToken) => + public Task PostAsync(string? requestUri, HttpContent? content, CancellationToken cancellationToken) => PostAsync(CreateUri(requestUri), content, cancellationToken); - public Task PostAsync(Uri? requestUri, HttpContent content, CancellationToken cancellationToken) + public Task PostAsync(Uri? requestUri, HttpContent? content, CancellationToken cancellationToken) { HttpRequestMessage request = CreateRequestMessage(HttpMethod.Post, requestUri); request.Content = content; return SendAsync(request, cancellationToken); } - public Task PutAsync(string? requestUri, HttpContent content) => + public Task PutAsync(string? requestUri, HttpContent? content) => PutAsync(CreateUri(requestUri), content); - public Task PutAsync(Uri? requestUri, HttpContent content) => + public Task PutAsync(Uri? requestUri, HttpContent? content) => PutAsync(requestUri, content, CancellationToken.None); - public Task PutAsync(string? requestUri, HttpContent content, CancellationToken cancellationToken) => + public Task PutAsync(string? requestUri, HttpContent? content, CancellationToken cancellationToken) => PutAsync(CreateUri(requestUri), content, cancellationToken); - public Task PutAsync(Uri? requestUri, HttpContent content, CancellationToken cancellationToken) + public Task PutAsync(Uri? requestUri, HttpContent? content, CancellationToken cancellationToken) { HttpRequestMessage request = CreateRequestMessage(HttpMethod.Put, requestUri); request.Content = content; return SendAsync(request, cancellationToken); } - public Task PatchAsync(string? requestUri, HttpContent content) => + public Task PatchAsync(string? requestUri, HttpContent? content) => PatchAsync(CreateUri(requestUri), content); - public Task PatchAsync(Uri? requestUri, HttpContent content) => + public Task PatchAsync(Uri? requestUri, HttpContent? content) => PatchAsync(requestUri, content, CancellationToken.None); - public Task PatchAsync(string? requestUri, HttpContent content, CancellationToken cancellationToken) => + public Task PatchAsync(string? requestUri, HttpContent? content, CancellationToken cancellationToken) => PatchAsync(CreateUri(requestUri), content, cancellationToken); - public Task PatchAsync(Uri? requestUri, HttpContent content, CancellationToken cancellationToken) + public Task PatchAsync(Uri? requestUri, HttpContent? content, CancellationToken cancellationToken) { HttpRequestMessage request = CreateRequestMessage(HttpMethod.Patch, requestUri); request.Content = content; -- 2.7.4