Add timeout support to AmazonS3Client

Author: mdupras

src/ImageSharp.Web.Providers.AWS/AmazonS3ClientFactory.cs

@@ -3,6 +3,7 @@
 
 using System;
 using Amazon;
+using Amazon.Runtime;
 using Amazon.S3;
 
 namespace SixLabors.ImageSharp.Web
@@ -25,6 +26,7 @@ public static AmazonS3Client CreateClient(IAWSS3BucketClientOptions options)
                 // AccessSecret can be empty.
                 // PathStyle endpoint doesn't support AccelerateEndpoint.
                 AmazonS3Config config = new() { ServiceURL = options.Endpoint, ForcePathStyle = true, AuthenticationRegion = options.Region };
+                SetTimeout(config, options.Timeout);
                 return new AmazonS3Client(options.AccessKey, options.AccessSecret, config);
             }
             else if (!string.IsNullOrWhiteSpace(options.AccessKey))
@@ -33,18 +35,29 @@ public static AmazonS3Client CreateClient(IAWSS3BucketClientOptions options)
                 Guard.NotNullOrWhiteSpace(options.Region, nameof(options.Region));
                 var region = RegionEndpoint.GetBySystemName(options.Region);
                 AmazonS3Config config = new() { RegionEndpoint = region, UseAccelerateEndpoint = options.UseAccelerateEndpoint };
+                SetTimeout(config, options.Timeout);
                 return new AmazonS3Client(options.AccessKey, options.AccessSecret, config);
             }
             else if (!string.IsNullOrWhiteSpace(options.Region))
             {
                 var region = RegionEndpoint.GetBySystemName(options.Region);
                 AmazonS3Config config = new() { RegionEndpoint = region, UseAccelerateEndpoint = options.UseAccelerateEndpoint };
+                SetTimeout(config, options.Timeout);
                 return new AmazonS3Client(config);
             }
             else
             {
                 throw new ArgumentException("Invalid configuration.", nameof(options));
             }
         }
+
+        private static void SetTimeout(ClientConfig config, TimeSpan? timeout)
+        {
+            // We don't want to override the default timeout if it's not set.
+            if (timeout.HasValue)
+            {
+                config.Timeout = timeout.Value;
+            }
+        }
     }
 }

src/ImageSharp.Web.Providers.AWS/Caching/AWSS3StorageCacheOptions.cs

@@ -25,5 +25,8 @@ public class AWSS3StorageCacheOptions : IAWSS3BucketClientOptions
 
         /// <inheritdoc/>
         public bool UseAccelerateEndpoint { get; set; }
+
+        /// <inheritdoc/>
+        public TimeSpan? Timeout { get; set; }
     }
 }

src/ImageSharp.Web.Providers.AWS/IAWSS3BucketClientOptions.cs

@@ -43,5 +43,11 @@ internal interface IAWSS3BucketClientOptions
         /// The feature must be enabled on the bucket. Follow AWS instruction on <see href="https://docs.aws.amazon.com/AmazonS3/latest/userguide/transfer-acceleration.html"/>.
         /// </summary>
         bool UseAccelerateEndpoint { get; set; }
+
+        /// <summary>
+        /// Gets or sets a value indicating the timeout for the S3 client.
+        /// If the value is set, the value is assigned to the Timeout property of the HttpWebRequest/HttpClient object used to send requests.
+        /// </summary>
+        TimeSpan? Timeout { get; set; }
     }
 }

src/ImageSharp.Web.Providers.AWS/Providers/AWSS3StorageImageProviderOptions.cs

@@ -38,5 +38,8 @@ public class AWSS3BucketClientOptions : IAWSS3BucketClientOptions
 
         /// <inheritdoc/>
         public bool UseAccelerateEndpoint { get; set; }
+
+        /// <inheritdoc/>
+        public TimeSpan? Timeout { get; set; }
     }
 }