Skip to content

fix(preprod): Enforce has_project_access on snapshot detail GET and DELETE#114169

Merged
michelletran-sentry merged 1 commit intomasterfrom
fix/preprod-snapshot-project-access
Apr 29, 2026
Merged

fix(preprod): Enforce has_project_access on snapshot detail GET and DELETE#114169
michelletran-sentry merged 1 commit intomasterfrom
fix/preprod-snapshot-project-access

Conversation

@michelletran-sentry
Copy link
Copy Markdown
Contributor

OrganizationPreprodSnapshotEndpoint resolved the artifact by project__organization_id but never checked whether the requesting member had access to the artifact's project. Add the has_project_access check on both handlers, mirroring the sibling PreprodArtifactEndpoint pattern with a staff bypass.

GH-20078

…ELETE

OrganizationPreprodSnapshotEndpoint resolved the artifact by
project__organization_id but never checked whether the requesting
member had access to the artifact's project. Add the
has_project_access check on both handlers, mirroring the sibling
PreprodArtifactEndpoint pattern with a staff bypass.

GH-20078
@michelletran-sentry michelletran-sentry requested a review from a team as a code owner April 28, 2026 14:57
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 28, 2026
@michelletran-sentry michelletran-sentry enabled auto-merge (squash) April 28, 2026 20:03
@michelletran-sentry michelletran-sentry merged commit 007d73b into master Apr 29, 2026
57 checks passed
@michelletran-sentry michelletran-sentry deleted the fix/preprod-snapshot-project-access branch April 29, 2026 13:48
cleptric pushed a commit that referenced this pull request May 5, 2026
…ELETE (#114169)

OrganizationPreprodSnapshotEndpoint resolved the artifact by
project__organization_id but never checked whether the requesting member
had access to the artifact's project. Add the has_project_access check
on both handlers, mirroring the sibling PreprodArtifactEndpoint pattern
with a staff bypass.

GH-20078
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