-
Notifications
You must be signed in to change notification settings - Fork 5k
SmtpClientSendMailTest_SendAsync.MultipleRecipients_Failure_All test failure #115070
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
Comments
Tagging subscribers to this area: @dotnet/ncl |
This seem weird, the client will attempt to send mail even if there are no valid recipients
|
The bug is due to runtime/src/libraries/System.Net.Mail/src/System/Net/Mail/SmtpTransport.cs Lines 315 to 334 in 215b39a
In case all sends finish synchronously, we proceed to SendData() without checking if at least one recipient succeeded. |
Failed in: runtime-coreclr libraries-jitstress 20250505.1 Failed tests:
Error message:
Stack trace:
|
Failed in: runtime-coreclr libraries-jitstressregs 20250511.1 Failed tests:
Error message:
Stack trace:
|
@rzikm Given that you have identified the issue, can you submit a fix or disable the test to make it unblocking? |
The bug should be fixed by #115722 |
Failed in: runtime-coreclr libraries-jitstress 20250522.1 Failed tests:
Error message:
Stack trace:
|
Failed in: runtime-coreclr libraries-jitstressregs 20250525.1 Failed tests:
Error message:
Stack trace:
|
Uh oh!
There was an error while loading. Please reload this page.
Build Information
Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=1026990
Build error leg or test failing: System.Net.Mail.Tests.SmtpClientSendMailTest_SendAsync.MultipleRecipients_Failure_All
Pull request: #115065
Error Message
Fill the error message using step by step known issues guidance.
Report
Summary
The text was updated successfully, but these errors were encountered: