-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Closed
Labels
area-System.Net.Httpbughelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributors
Milestone
Description
(based on code inspection)
The SendPingAsync call that we do as part of the ConnectionPoolManager's heartbeat isn't awaited and so any exceptions from it would bubble up as UnobservedTaskExceptions.
runtime/src/libraries/System.Net.Http/src/System/Net/Http/SocketsHttpHandler/Http2Connection.cs
Line 2041 in 82c45ca
| SendPingAsync(pingPayload); |
We should consume the task with LogExceptions(SendPingAsync(pingPayload)) as we do when sending out pings normally.
Should be an easy fix - marking as up for grabs.
ManickaP and antonfirsov
Metadata
Metadata
Assignees
Labels
area-System.Net.Httpbughelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributors