[Analyzer] Async methods that don't take a CancellationToken

[stephentoub]

Cancellation is often important to the scalability of an app / service. If an async method has no way to cancel it, that can impact scalability and responsiveness. We should consider adding an analyzer (likely off by default due to noise) that flags async methods (async and/or Task/ValueTask-returning) that have no overloads which accept a CancellationToken, and issue a diagnostic to consider taking one.

[ghost]

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

Issue Details

---

Cancellation is often important to the scalability of an app / service. If an async method has no way to cancel it, that can impact scalability and responsiveness. We should consider adding an analyzer (likely off by default due to noise) that flags async methods (async and/or Task/ValueTask-returning) that have no overloads which accept a CancellationToken, and issue a diagnostic to consider taking one.

Author:	stephentoub
Assignees:	-
Labels:	area-System.Threading.Tasks, code-analyzer
Milestone:	-

[bartonjs]

Video

Looks good as proposed (finding Task-like returning method groups with no overloads that accept a CancellationToken).

Category: Design
Severity: None (off by default)

[pedrodeandrade]

I would like to be assigned to this, please

[buyaa-n]

Thanks @pedrodeandrade, assigned to you