-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[release/8.0] Added missing *Async overrides to TlsStream #91794
Conversation
Tagging subscribers to this area: @dotnet/ncl Issue DetailsBackport of #91750 to release/8.0 Customer ImpactTestingRiskIMPORTANT: If this backport is for a servicing release, please verify that:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did one last round of verification on the fix and the repro-case passes with this change.
@artl93 this is ready for your approval -- it is a regression in 8.0. |
M2 approved. |
Backport of #91750 to release/8.0
Fixes #91377
/cc @stephentoub @ManickaP
Customer Impact
Regression against 7.0. Unable to send email via
SmptClient
over TLS.Testing
Manual test.
Note: Adding automated test is involved work - tracked in #91776.
Risk
Low