Skip to content

Check current state for user and OAuth tokens - #2634

Draft
niemyjski wants to merge 14 commits into
mainfrom
issue/current-user-token-state
Draft

niemyjski wants to merge 14 commits into
mainfrom
issue/current-user-token-state

Conversation

@niemyjski

@niemyjski niemyjski commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

This draft checks stored user-session and OAuth token state after logout or revocation. It now depends directly on #2627 rather than inheriting the unrelated password and concurrency proposals.

BLOCKER: the current approach can read a user token twice on a cache miss, and uncached OAuth application lookups can trigger Elasticsearch refreshes per request. A bounded, high-throughput lookup contract and corresponding regression coverage are required before merging.

Validation: production changes rebased onto the updated session-generation implementation. Endpoint runtime verification remains blocked by Aspire/DCP startup. This PR is not ready to merge.

@niemyjski
niemyjski force-pushed the issue/password-check-capacity branch from 215c37e to d8cefff Compare October 13, 2026 02:39
@niemyjski
niemyjski force-pushed the issue/current-user-token-state branch 2 times, most recently from 76e8282 to 229c8d7 Compare October 13, 2026 02:41
@niemyjski
niemyjski force-pushed the issue/password-check-capacity branch from d8cefff to 79578cc Compare October 13, 2026 02:41
@niemyjski
niemyjski force-pushed the issue/current-user-token-state branch from 229c8d7 to 1ff975e Compare October 13, 2026 02:58
@niemyjski
niemyjski force-pushed the issue/password-check-capacity branch from 79578cc to 121c87b Compare October 13, 2026 02:58
@niemyjski
niemyjski force-pushed the issue/current-user-token-state branch from 1ff975e to 5601ad2 Compare October 13, 2026 03:05
@niemyjski
niemyjski force-pushed the issue/password-check-capacity branch from 121c87b to 3bdef04 Compare October 13, 2026 03:05
@niemyjski
niemyjski added this pull request to stack #2659 October 13, 2026 03:05
@niemyjski
niemyjski removed this pull request from stack #2659 October 13, 2026 12:26
@niemyjski
niemyjski force-pushed the issue/password-check-capacity branch from 3bdef04 to e42d7f5 Compare October 13, 2026 12:26
@niemyjski
niemyjski force-pushed the issue/current-user-token-state branch from 5601ad2 to 609782b Compare October 13, 2026 12:26
@niemyjski
niemyjski changed the base branch from issue/password-check-capacity to issue/user-session-generation October 13, 2026 12:27
@niemyjski
niemyjski force-pushed the issue/current-user-token-state branch 2 times, most recently from f610a5c to 5494433 Compare October 13, 2026 13:33
@github-actions

Copy link
Copy Markdown

Code Coverage

Package Line Rate Branch Rate Complexity Health
Exceptionless.AppHost 23% 23% 128 ❌
Exceptionless.Core 77% 69% 10881 ✔
Exceptionless.Insulation 51% 43% 370 ➖
Exceptionless.Web 86% 71% 9189 ✔
Summary 80% (28134 / 35112) 69% (13904 / 20148) 20568 ✔

@niemyjski
niemyjski force-pushed the issue/current-user-token-state branch from 5494433 to 137f849 Compare October 13, 2026 13:55
@niemyjski
niemyjski force-pushed the issue/user-session-generation branch 3 times, most recently from 9593b05 to 2ef6152 Compare October 14, 2026 02:38
@niemyjski
niemyjski force-pushed the issue/current-user-token-state branch 2 times, most recently from 2128046 to a3a453e Compare October 14, 2026 02:50
@niemyjski
niemyjski force-pushed the issue/user-session-generation branch from 4222af7 to a178e7b Compare October 14, 2026 03:03
@niemyjski
niemyjski force-pushed the issue/current-user-token-state branch from a3a453e to 21eef5f Compare October 14, 2026 03:03
Base automatically changed from issue/user-session-generation to main October 14, 2026 03:16

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