Skip to content

Use current account state during password sign-in - #2632

Draft
niemyjski wants to merge 1 commit into
mainfrom
issue/password-current-state
Draft

niemyjski wants to merge 1 commit into
mainfrom
issue/password-current-state

Conversation

@niemyjski

@niemyjski niemyjski commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

Password authentication now checks the current stored credentials, email and active state through the existing AuthService. This closes stale cached-user authentication after account changes and is independent of the session-generation and optimistic-concurrency proposals, targeting main directly.

Validation: build passed with zero warnings/errors; 63 focused tests passed. Five controlled mutation cases failed when the fresh read was removed. Endpoint verification was blocked before assertions by Aspire/DCP KubernetesService startup timing out after 60 seconds. Fresh reads do not protect against stale writers overwriting the database itself.

@niemyjski
niemyjski added this pull request to stack #2642 October 5, 2026 02:11
@niemyjski
niemyjski force-pushed the issue/password-current-state branch from 52bf4e7 to 5b84a9c Compare October 13, 2026 02:39
@niemyjski
niemyjski force-pushed the issue/user-record-concurrency branch from 179c281 to 032664f Compare October 13, 2026 02:39
@niemyjski
niemyjski force-pushed the issue/password-current-state branch from 5b84a9c to dbeb9b2 Compare October 13, 2026 02:41
@niemyjski
niemyjski force-pushed the issue/user-record-concurrency branch 2 times, most recently from 57f409f to 975b25e Compare October 13, 2026 02:58
@niemyjski
niemyjski force-pushed the issue/password-current-state branch from dbeb9b2 to 7592a6f Compare October 13, 2026 02:58
@niemyjski
niemyjski removed this pull request from stack #2642 October 13, 2026 03:05
@niemyjski
niemyjski force-pushed the issue/user-record-concurrency branch from 975b25e to 793b7c9 Compare October 13, 2026 03:05
@niemyjski
niemyjski force-pushed the issue/password-current-state branch from 7592a6f to e49585d Compare October 13, 2026 03:05
@niemyjski
niemyjski added this pull request to stack #2659 October 13, 2026 03:05
@github-actions

Copy link
Copy Markdown

Code Coverage

Package Line Rate Branch Rate Complexity Health
Exceptionless.AppHost 23% 23% 128 ❌
Exceptionless.Core 77% 69% 10969 ✔
Exceptionless.Insulation 51% 43% 370 ➖
Exceptionless.Web 86% 71% 9220 ✔
Summary 80% (28303 / 35267) 69% (14017 / 20270) 20687 ✔

@niemyjski
niemyjski removed this pull request from stack #2659 October 13, 2026 12:26
@niemyjski
niemyjski force-pushed the issue/password-current-state branch from e49585d to bde0c24 Compare October 13, 2026 12:26
@niemyjski
niemyjski force-pushed the issue/user-record-concurrency branch from 793b7c9 to 2c36ec1 Compare October 13, 2026 12:26
@niemyjski
niemyjski changed the base branch from issue/user-record-concurrency to main October 13, 2026 12:26

This branch has not been deployed

No deployments
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.

1 participant