Skip to content

feat(service-accounts): Require service_account:read for service account reads - #129497

Merged
skaasten merged 2 commits into
masterfrom
feat/service-account-read-scope
Nov 2, 2026
Merged

skaasten merged 2 commits into
masterfrom
feat/service-account-read-scope

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Service account list and detail reads now require service_account:read. Member, Admin, Manager, and Owner receive the scope, preserving their access; write permissions are unchanged.

via shaun.kaasten.

--

View Session [Sentry]

…unt reads

List and detail now require service_account:read instead of org:read.
Manager and Owner hold the new scope, so members can no longer list
service accounts. The scope is explicit: org:read and
service_account:write do not grant reads, so tokens that read and write
need both service account scopes.

Co-Authored-By: Shaun Kaasten <shaun.kaasten@sentry.io>
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 2, 2026
Members should be able to list service accounts, for example to pick one
for a Seer automation. Member and Admin now hold service_account:read, so
moving reads off org:read keeps their access. Writes stay limited to
Manager and Owner.

Co-Authored-By: Shaun Kaasten <shaun.kaasten@sentry.io>
@sentry-junior
sentry-junior Bot requested review from a team as code owners November 2, 2026 17:28
@skaasten
skaasten merged commit d0bb583 into master Nov 2, 2026
99 checks passed
@skaasten
skaasten deleted the feat/service-account-read-scope branch November 2, 2026 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants