-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MsQuic configuration handle is created but not used #55158
Comments
Tagging subscribers to this area: @dotnet/ncl Issue Details
|
Note also, we create another equivalent config handle here: https://github.com/dotnet/runtime/blob/main/src/libraries/System.Net.Quic/src/System/Net/Quic/Implementations/MsQuic/MsQuicConnection.cs#L162 This one is stored into a field, then used later in ConnectAsync. This all seems less than ideal. I wonder if we should just pass QuicClientConnectionOptions to ConnectAsync, or something like that. |
Relevant issue: #42642 |
Triage: Not needed for 6.0, but super simple (remove 1 line). |
See https://github.com/dotnet/runtime/blob/main/src/libraries/System.Net.Quic/src/System/Net/Quic/Implementations/MsQuic/MsQuicConnection.cs#L175
The text was updated successfully, but these errors were encountered: