Feature request: Enable Websocket ping-pong and timeout

[zackliu]

Currently, ManagedWebsocket only sends Pong to keep live. It means it have to depend on TCP timeout to terminate a half-open connection. TCP timeout depends on OS and it's around 15 minutes in Linux, which is too long especially for the Websocket server side.

Although RFC is not described clearly about what should do if no pong received after a period of time, I suggest we can have an option to enable a timeout.

The purpose here is to make sure the other side of connection is still alive, so as long as the connection can receive some bytes, we can treat it as alive. Sending ping is just force the other side to send something back.

I propose sending Ping instead of Pong periodically here:
https://github.com/dotnet/runtime/blob/main/src/libraries/System.Net.WebSockets/src/System/Net/WebSockets/ManagedWebSocket.cs#L546
And adding another periodically check to verify whether the connection not received anything for too long.

[ghost]

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

Issue Details

---

Currently, ManagedWebsocket only sends Pong to keep live. It means it have to depend on TCP timeout to terminate a half-open connection. TCP timeout depends on OS and it's around 15 minutes in Linux, which is too long especially for the Websocket server side.

Although RFC is not described clearly about what should do if no pong received after a period of time, I suggest we can have an option to enable a timeout.

The purpose here is to make sure the other side of connection is still alive, so as long as the connection can receive some bytes, we can treat it as alive. Sending ping is just force the other side to send something back.

I propose sending Ping instead of Pong periodically here:
https://github.com/dotnet/runtime/blob/main/src/libraries/System.Net.WebSockets/src/System/Net/WebSockets/ManagedWebSocket.cs#L546
And adding another periodically check to verify whether the connection not received anything for too long.

Author:	zackliu
Assignees:	-
Labels:	area-System.Net, untriaged
Milestone:	-

[CarnaViire]

Thanks @zackliu. This feature request is actually a duplicate of #48729, so I'm going to close this one.

If you are interested in the feature, please upvote #48729, it would help us prioritize!

[CarnaViire]

Duplicate of #48729