Set SocketsHttpHandler's default connect timeout to 15s (#66607)

* Set default connect timeout to 15s * Fix ConnectTimeout_Default test
dotnet · Mar 15, 2022 · 436b97c · 436b97c
1 parent 734362c
commit 436b97c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/libraries/Common/src/System/Net/Http/HttpHandlerDefaults.cs b/src/libraries/Common/src/System/Net/Http/HttpHandlerDefaults.cs
@@ -27,6 +27,6 @@ internal static partial class HttpHandlerDefaults
         public static readonly TimeSpan DefaultPooledConnectionLifetime = Timeout.InfiniteTimeSpan;
         public static readonly TimeSpan DefaultPooledConnectionIdleTimeout = TimeSpan.FromMinutes(1);
         public static readonly TimeSpan DefaultExpect100ContinueTimeout = TimeSpan.FromSeconds(1);
-        public static readonly TimeSpan DefaultConnectTimeout = Timeout.InfiniteTimeSpan;
+        public static readonly TimeSpan DefaultConnectTimeout = TimeSpan.FromSeconds(15);
     }
 }
diff --git a/src/libraries/System.Net.Http/tests/FunctionalTests/SocketsHttpHandlerTest.cs b/src/libraries/System.Net.Http/tests/FunctionalTests/SocketsHttpHandlerTest.cs
@@ -1054,7 +1054,7 @@ public void ConnectTimeout_Default()
         {
             using (var handler = new SocketsHttpHandler())
             {
-                Assert.Equal(Timeout.InfiniteTimeSpan, handler.ConnectTimeout);
+                Assert.Equal(TimeSpan.FromSeconds(15), handler.ConnectTimeout);
             }
         }