throw PNSE for unsupported SSL options in Quic.#55877
Conversation
|
Tagging subscribers to this area: @dotnet/ncl Issue Detailsthrow for few options we do not support to make it obvious fixes #55759
|
PNSP -> PNSE? |
|
You've added the validation into |
|
ok, that make sense. I will update #55468 as well since I'm going to probably touch it anyway. We may split pre-validation to separate method but I'm not sure I want to do that right now. ConfigurationOpenDelegate may still fail so we should figure out how to better surface exceptions. |
ManickaP
left a comment
There was a problem hiding this comment.
LGTM, thanks. Looking forward for the SNI PR solution.
...Net.Quic/src/System/Net/Quic/Implementations/MsQuic/Interop/SafeMsQuicConfigurationHandle.cs
Outdated
Show resolved
Hide resolved
|
failing test fixed by #56047. |
throw for few options we do not support to make it obvious
fixes #55759