Remove some IAsyncResult implementations from System.Net.Mail

[stephentoub]

Whittling away at our remaining APM implementations.

[ghost]

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Whittling away at our remaining APM implementations.

Author:	stephentoub
Assignees:	-
Labels:	area-System.Net
Milestone:	-

[wfurt]

nice. Do you plan to do rest of the Mail as well? (at some point)

[stephentoub]

Ideally. The rest is harder and more error-prone, though, so I'm hesitant to attempt it with our current test coverage.

[wfurt]

It has been on my wishlist for a long time. Maybe @filipnavara or others can help. We did talk about #19436 specifically but I'm sure there may be more.

[filipnavara]

It has been on my wishlist for a long time. Maybe @filipnavara or others can help. We did talk about #19436 specifically but I'm sure there may be more.

This one is very low on my list given the fact that the whole library is effectively deprecated. I have some cleanup PRs for System.Net.Security that I would like to resurrect in the .NET 8 timeframe but currently nothing for the Mail libraries.

[stephentoub]

I have some cleanup PRs for System.Net.Security

There aren't any APM implementations remaining there, is there?

[filipnavara]

I have some cleanup PRs for System.Net.Security

There aren't any APM implementations remaining there, is there?

There's NegotiateStream but it already uses the TaskToAsyncResult pattern in all the places IIRC. I didn't mean that my cleanups are specific to APM. Sorry, should have made that clearer.