Concurrency and State Risk

Why this rule exists

async void cannot be awaited and crashes the process on unhandled exceptions. Blocking on async in a SynchronizationContext deadlocks under load. lock(this) exposes the lock to callers. Thread.Sleep blocks thread-pool threads. Comment/string false positives are stripped before matching.

Code example

+ public async void HandleClick() { await SaveAsync(); }
+ var data = httpClient.GetStringAsync(url).Result;

+ public async Task HandleClickAsync() { await SaveAsync(); }
+ var data = await httpClient.GetStringAsync(url);

Configuration

Disable or adjust the severity of this rule in .gauntletci.json:

{
  "rules": {
    "GCI0016": { "enabled": true, "severity": "Block" }
  }
}

See Configuration for the full schema.

Related rules

Discussed in