Skip to content

v8.1.13: P1-3 DoLoginAsync — eliminate ThreadPool starvation#7

Merged
cct08311github merged 1 commit intodotnet8from
feature/8.1.13-p1-3-async-login
Mar 3, 2026
Merged

v8.1.13: P1-3 DoLoginAsync — eliminate ThreadPool starvation#7
cct08311github merged 1 commit intodotnet8from
feature/8.1.13-p1-3-async-login

Conversation

@cct08311github
Copy link
Copy Markdown
Owner

Summary

  • P1-3 (Issue [P1-3] Fix sync-over-async patterns (ThreadPool starvation) #5): Add DoLoginAsync to WTMContext — fully async implementation replacing 6 .Result/.Wait() blocking calls on every authentication request.
  • Mark DoLogin as [Obsolete], kept as a GetAwaiter().GetResult() backward-compat wrapper.
  • DC.SaveChanges()DC.SaveChangesAsync() in the PBKDF2 auto-rehash path.
  • Update all 5 demo AccountController + LoginController callers to await DoLoginAsync(...).

Test plan

Closes #5

🤖 Generated with Claude Code

…y login

Fixes #5

- Add DoLoginAsync with full async/await implementation (6 former .Result/.Wait() calls)
- DC.SaveChanges() → SaveChangesAsync() for MD5 auto-rehash path
- Mark DoLogin [Obsolete], keep as GetAwaiter().GetResult() wrapper for backward compat
- Update all 5 demo AccountControllers + LoginController to await DoLoginAsync

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cct08311github cct08311github merged commit 0bb5201 into dotnet8 Mar 3, 2026
3 checks passed
@cct08311github cct08311github deleted the feature/8.1.13-p1-3-async-login branch March 3, 2026 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[P1-3] Fix sync-over-async patterns (ThreadPool starvation)

1 participant