Skip to content

CI run: org list rows for whole-org keys (not for merge) - #2

Closed
jbob06 wants to merge 1 commit into
mainfrom
audit-org-lists-for-keys
Closed

jbob06 wants to merge 1 commit into
mainfrom
audit-org-lists-for-keys

Conversation

@jbob06

@jbob06 jbob06 commented Sep 26, 2026

Copy link
Copy Markdown
Owner

Opened only so the database suites run in this fork's Actions before the upstream PR. Will be closed.

…anges

An org ban or reserve list change is logged on the org with no server
(serverId null), and auditVisibility gives an API key no owned orgs, so no
key sees one, whatever its scope. A key that reads the audit trail of every
server in the org, which is what an integration watching the whole org
holds, missed exactly the changes those servers all enforce: an org-wide ban
reached every server and never reached the key's audit read.

A key with audit.read held to every server of its org (serverIds null) now
also sees that org's list-entry changes: list.add, list.remove,
list.update, list.import and list.expire (ORG_LIST_ROWS). Nothing else
logged on the org: member grants, roles, keys, webhooks and list.members
stay the owners'. A key held to some servers, a key without audit.read, a
key of a suspended org, and every person's visibility are unchanged.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011wkaN3kHsCQGhxCxeAqERQ
@jbob06

jbob06 commented Sep 26, 2026

Copy link
Copy Markdown
Owner Author

CI passed (848 pass, 0 fail); the real PR is upstream.

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