Reduce DefaultPooledConnectionIdleTimeout default

[halter73]

• Change DefaultPooledConnectionIdleTimeout from 120 seconds to 60 seconds
• This should reduce the occurrence of socket errors near the timeout when connected to IIS or Kestrel

See #52267 (comment)

Here's the similar PR for Kestrel to increase its KeepAliveTimeout: dotnet/aspnetcore#32636

Other servers have even smaller default keep-alive timeouts, but I still think this is a good change.

• Node.js 5s
• Apache 5s?
• NGINX 75s

We'll need to update the API docs at https://github.com/dotnet/dotnet-api-docs/blob/2238750cc696339f0a8f04c786f7fe03b14e51ec/xml/System.Net.Http/SocketsHttpHandler.xml#L623

Addresses #52267

@geoffkizer

[ghost]

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

---

• Change DefaultPooledConnectionIdleTimeout from 120 seconds to 110 seconds
• This should reduce the occurrence of socket errors near the timeout when connected to IIS or Kestrel

See #52267 (comment)

Here's the similar PR for Kestrel to increase its KeepAliveTimeout: dotnet/aspnetcore#32636

We'll need to update the API docs at https://github.com/dotnet/dotnet-api-docs/blob/2238750cc696339f0a8f04c786f7fe03b14e51ec/xml/System.Net.Http/SocketsHttpHandler.xml#L623

Addresses #52267

@geoffkizer

Author:	halter73
Assignees:	-
Labels:	area-System.Net.Http
Milestone:	-

[geoffkizer]

Why not make it even smaller, e.g. 60 seconds? 110 is a weird number. 60 seconds seems like plenty.

[halter73]

I changed it to 60 seconds.

[geoffkizer]

@scalablecory @karelz @dotnet/ncl Any opinions/concerns here?

I think this is a reasonable change.

Note this timeout matches WinHttp, it also uses 60s as the default.

[wfurt]

I think it is ok. We would generally see more handshakes for anybody with 1m repetition.

[karelz]

Personally I am fine with either 110 or 60 (small preference to 110 as it is closer to original value, but not significant preference). Up to others to decide.

[stephentoub]

Seems reasonable to me.

[ManickaP]

We should update the docs, it still contains 2 minutes as default: https://docs.microsoft.com/en-us/dotnet/api/system.net.http.socketshttphandler.pooledconnectionidletimeout?view=net-6.0

EDIT: dotnet/dotnet-api-docs#7449