[automated] Merge branch 'release/8.0.2xx' => 'release/8.0.3xx' (#39407)

Author: v-wuzhai

src/Containers/Microsoft.NET.Build.Containers/Registry/DefaultRegistryAPI.cs

@@ -14,6 +14,14 @@ internal class DefaultRegistryAPI : IRegistryAPI
     private readonly HttpClient _client;
     private readonly ILogger _logger;
 
+    // Empirical value - Unoptimized .NET application layers can be ~200MB
+    // * .NET Runtime (~80MB)
+    // * ASP.NET Runtime (~25MB)
+    // * application and dependencies - variable, but _probably_ not more than the BCL?
+    // Given a 200MB target and a 1Mb/s upload speed, we'd expect an upload speed of 27m:57s.
+    // Making this a round 30 for convenience.
+    private static TimeSpan LongRequestTimeout = TimeSpan.FromMinutes(30);
+
     internal DefaultRegistryAPI(string registryName, Uri baseUri, ILogger logger)
     {
         bool isAmazonECRRegistry = baseUri.IsAmazonECRRegistry();
@@ -33,6 +41,8 @@ private static HttpClient CreateClient(string registryName, Uri baseUri, ILogger
         var innerHandler = new SocketsHttpHandler()
         {
             UseCookies = false,
+            // the rest of the HTTP stack has an very long timeout (see below) but we should still have a reasonable timeout for the initial connection
+            ConnectTimeout = TimeSpan.FromSeconds(30)
         };
 
         // Ignore certificate for https localhost repository.
@@ -52,7 +62,10 @@ private static HttpClient CreateClient(string registryName, Uri baseUri, ILogger
             clientHandler = new AmazonECRMessageHandler(clientHandler);
         }
 
-        HttpClient client = new(clientHandler);
+        HttpClient client = new(clientHandler)
+        {
+            Timeout = LongRequestTimeout
+        };
 
         client.DefaultRequestHeaders.Add("User-Agent", $".NET Container Library v{Constants.Version}");