Skip to content

test(seer): Add service account auth matrix - #128814

Merged
gricha merged 1 commit into
masterfrom
feat/sa-auth-matrix
Oct 27, 2026
Merged

gricha merged 1 commit into
masterfrom
feat/sa-auth-matrix

Conversation

@gricha

@gricha gricha commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Adds service account ViewerContext rows to the agent-token auth matrix, so we can iterate on service account authorization endpoint by endpoint.

Oracle: an owner token holding the account's scopes. The account is linked to the project in an open-membership org, so it should reach everything the owner does. User-global routes must deny it.

Until service account auth lands (#128779), every row is a non-strict xfail. After that, rows that still don't match are listed in SERVICE_ACCOUNT_KNOWN_GAPS as strict xfails, with the cause as the reason. A fixed row XPASSes and fails until its entry is removed. Against #128779, 36 of 261 rows are gaps:

Cause Rows
Endpoint requires or stores state for request.user 19
Control silo (SA auth is cell-only) 12
500 using an anonymous request.user as an actor 3
Checks the caller's org role 2

@gricha gricha changed the title test(seer): Add opt-in service account auth matrix test(seer): Add service account auth matrix Oct 26, 2026
@gricha
gricha force-pushed the feat/sa-auth-matrix branch 2 times, most recently from f4a7f5d to 2ddc384 Compare October 27, 2026 02:37
@gricha
gricha changed the base branch from feat/service-account-actor to master October 27, 2026 02:37
@gricha
gricha marked this pull request as ready for review October 27, 2026 16:28
@gricha
gricha requested a review from a team as a code owner October 27, 2026 16:28
Service account ViewerContext rows in the agent-token auth matrix. Every row is xfail until service account auth lands. After that, rows that don't match the owner-token oracle yet are strict xfail, with the cause as the reason.
@gricha
gricha force-pushed the feat/sa-auth-matrix branch from 2ddc384 to e9e1823 Compare October 27, 2026 16:28
@gricha
gricha merged commit 468fd01 into master Oct 27, 2026
69 checks passed
@gricha
gricha deleted the feat/sa-auth-matrix branch October 27, 2026 17:30
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.

2 participants