Skip to content

fix(api): Allow unlinking external issues with event:write - #123925

Open
betegon wants to merge 1 commit into
masterfrom
bt/fix-issue-unlink-write
Open

fix(api): Allow unlinking external issues with event:write#123925
betegon wants to merge 1 commit into
masterfrom
bt/fix-issue-unlink-write

Conversation

@betegon

@betegon betegon commented Sep 9, 2026

Copy link
Copy Markdown
Member

Allow event:write to unlink existing external issues through native integrations and both Sentry App DELETE routes. Users who can link a resource can also unlink it, including project members whose organization disables eventsMemberAdmin.

The group routes use a narrow GroupPermission subclass; the installation route updates its existing permission map. event:admin remains accepted, and deleting an actual Sentry issue still requires it. Existing organization, project and App identity checks remain in place. Paths, link deletion behavior and RPC contracts are unchanged.

There are existing precedents for DELETE with write permission:

The issue PUT and native integration PUT already accept event:write. This extends that policy specifically to removing external references; the shared issue DELETE permission is unchanged.

Accept write or admin for external issue unlink while preserving project and organization checks.

Keep actual issue deletion restricted to event:admin.
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 9, 2026
@betegon
betegon marked this pull request as ready for review September 9, 2026 18:13
@betegon
betegon requested review from a team as code owners September 9, 2026 18:13
@betegon

betegon commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

The optional openapi-diff check exits 1 on the intended event:write scope additions. Schema generation and API docs tests pass.

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.

1 participant