Merge timeout

Author: alexeyzimarev

src/RestSharp/Options/RestClientOptions.cs

@@ -180,10 +180,9 @@ public RestClientOptions(string baseUrl) : this(new Uri(Ensure.NotEmptyString(ba
     public CookieContainer? CookieContainer { get; set; }
 
     /// <summary>
-    /// Maximum request duration in milliseconds. When the request timeout is specified using <seealso cref="RestRequest.Timeout"/>,
-    /// the lowest value between the client timeout and request timeout will be used.
+    /// Request duration. Used when the request timeout is not specified using <seealso cref="RestRequest.Timeout"/>,
     /// </summary>
-    public int MaxTimeout { get; set; }
+    public TimeSpan? Timeout { get; set; }
 
     /// <summary>
     /// Default encoding to use when no encoding is specified in the content type header.

src/RestSharp/Request/RestRequest.cs

@@ -139,7 +139,7 @@ public RestRequest(Uri resource, Method method = Method.Get)
     /// <summary>
     /// Custom request timeout
     /// </summary>
-    public int Timeout { get; set; }
+    public TimeSpan? Timeout { get; set; }
 
     /// <summary>
     /// The Resource URL to make the request against.

src/RestSharp/RestClient.Async.cs

@@ -19,6 +19,8 @@
 namespace RestSharp;
 
 public partial class RestClient {
+    // Default HttpClient timeout 
+    public TimeSpan DefaultTimeout = TimeSpan.FromSeconds(100);
     /// <inheritdoc />
     public async Task<RestResponse> ExecuteAsync(RestRequest request, CancellationToken cancellationToken = default) {
         using var internalResponse = await ExecuteRequestAsync(request, cancellationToken).ConfigureAwait(false);
@@ -113,7 +115,7 @@ async Task<HttpResponse> ExecuteRequestAsync(RestRequest request, CancellationTo
         message.Headers.Host         = Options.BaseHost;
         message.Headers.CacheControl = request.CachePolicy ?? Options.CachePolicy;
 
-        using var timeoutCts = new CancellationTokenSource(request.Timeout > 0 ? request.Timeout : int.MaxValue);
+        using var timeoutCts = new CancellationTokenSource(request.Timeout ?? Options.Timeout ?? DefaultTimeout);
         using var cts        = CancellationTokenSource.CreateLinkedTokenSource(timeoutCts.Token, cancellationToken);
 
         var ct = cts.Token;

src/RestSharp/RestClient.cs

@@ -226,7 +226,9 @@ public RestClient(
         : this(new HttpClient(handler, disposeHandler), true, configureRestClient, configureSerialization) { }
 
     static void ConfigureHttpClient(HttpClient httpClient, RestClientOptions options) {
-        if (options.MaxTimeout > 0) httpClient.Timeout = TimeSpan.FromMilliseconds(options.MaxTimeout);
+        
+        // We will use Options.Timeout in ExecuteAsInternalAsync method
+        httpClient.Timeout = Timeout.InfiniteTimeSpan;
 
         if (options.Expect100Continue != null) httpClient.DefaultRequestHeaders.ExpectContinue = options.Expect100Continue;
     }

test/RestSharp.Tests.Integrated/NonProtocolExceptionHandlingTests.cs

@@ -45,7 +45,7 @@ public async Task Handles_HttpClient_Timeout_Error() {
     public async Task Handles_Server_Timeout_Error() {
         using var client = new RestClient(_server.Url!);
 
-        var request  = new RestRequest("timeout") { Timeout = 500 };
+        var request  = new RestRequest("404") { Timeout = TimeSpan.FromMilliseconds(500) };
         var response = await client.ExecuteAsync(request);
 
         response.ErrorException.Should().BeOfType<TaskCanceledException>();
@@ -56,7 +56,7 @@ public async Task Handles_Server_Timeout_Error() {
     public async Task Handles_Server_Timeout_Error_With_Deserializer() {
         using var client = new RestClient(_server.Url!);
 
-        var request  = new RestRequest("timeout") { Timeout = 500 };
+        var request  = new RestRequest("timeout") { Timeout = TimeSpan.FromMilliseconds(500) };
         var response = await client.ExecuteAsync<SuccessResponse>(request);
 
         response.Data.Should().BeNull();

test/RestSharp.Tests.Integrated/PutTests.cs

@@ -31,7 +31,7 @@ public async Task Can_Timeout_PUT_Async() {
         var request = new RestRequest("/timeout", Method.Put).AddBody("Body_Content");
 
         // Half the value of ResponseHandler.Timeout
-        request.Timeout = 200;
+        request.Timeout = TimeSpan.FromMilliseconds(200);
 
         var response = await _client.ExecuteAsync(request);
 

test/RestSharp.Tests.Integrated/RequestTests.cs

@@ -55,7 +55,7 @@ public async Task Can_Timeout_GET_Async() {
         var request = new RestRequest("timeout").AddBody("Body_Content");
 
         // Half the value of ResponseHandler.Timeout
-        request.Timeout = 200;
+        request.Timeout = TimeSpan.FromMilliseconds(200);
 
         var response = await _client.ExecuteAsync(request);