Skip to content

docs(arch): say which requests skip the credential-epoch check - #1127

Closed
mforce wants to merge 1 commit into
mainfrom
docs/architecture-epoch-scope
Closed

mforce wants to merge 1 commit into
mainfrom
docs/architecture-epoch-scope

Conversation

@mforce

@mforce mforce commented Oct 7, 2026

Copy link
Copy Markdown
Owner

docs/architecture.md said the credential-epoch check reads the database on every authenticated request. CredentialEpochMiddleware skips two kinds of request: /api/v1/auth/logout, with or without a trailing slash, and the /error re-execution, which it detects through IExceptionHandlerFeature. The sentence now names both. It also says that a token whose user or account claim does not parse is rejected without a database read.

CredentialEpochFreshReadTests.EveryAuthenticatedRequest_ReadsTheCredentialRowOnce_AndOnlyLogoutIsExempt pins this behaviour.

This is a review finding (P3) on #1126 that arrived after #1126 merged.

Checks at the head: CouplingMatrixRealTreeTests and TenancyDocsFreshnessTests pass 4/4. The SchemaDocsTests image-pin guards and CredentialEpochFreshReadTests pass 11/11 together.

@mforce

mforce commented Oct 7, 2026

Copy link
Copy Markdown
Owner Author

Folded into #1128 at the owner's request.

@mforce mforce closed this Oct 7, 2026
@mforce
mforce deleted the docs/architecture-epoch-scope branch October 7, 2026 00:33
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