Skip to content

fix(difs): Clone Objectstore-backed Proguard DIFs correctly - #121050

Open
lcian wants to merge 4 commits into
feat/objectstore-debugfiles-assemblefrom
feat/proguard-objectstore-clones
Open

fix(difs): Clone Objectstore-backed Proguard DIFs correctly#121050
lcian wants to merge 4 commits into
feat/objectstore-debugfiles-assemblefrom
feat/proguard-objectstore-clones

Conversation

@lcian

@lcian lcian commented Aug 3, 2026

Copy link
Copy Markdown
Member

When an Objectstore copy exists, the ProGuard clone path now copies those bytes instead of being tied to the -read flag.
We still call into the newly updated create_dif_from_id that determines what kind of ProjectDebugFile to create depending on the active flags, making this consistent with the rest of the ProjectDebugFile creation methods.

Use the stored Objectstore copy when available so cloned DIFs follow the active double-write and exclusive-write modes.
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 3, 2026
@lcian lcian changed the title fix(difs): Clone ProGuard DIFs through Objectstore feat(difs): Clone ProGuard DIFs through Objectstore Aug 3, 2026
@lcian lcian changed the title feat(difs): Clone ProGuard DIFs through Objectstore fix(difs): Clone Objectstore-backed Proguard DIFs correctly regardless of read flag Aug 3, 2026
@lcian lcian changed the title fix(difs): Clone Objectstore-backed Proguard DIFs correctly regardless of read flag fix(difs): Clone Objectstore-backed Proguard DIFs correctly Aug 3, 2026
@lcian
lcian marked this pull request as ready for review August 3, 2026 11:19
@lcian
lcian requested review from a team as code owners August 3, 2026 11:19
@lcian
lcian requested a review from a team August 3, 2026 11:19
@sentry

sentry Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Sentry Snapshot Testing

Name Added Removed Changed Renamed Unchanged Skipped Status
sentry-frontend
sentry-frontend
0 0 0 0 451 0 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

Comment thread src/sentry/api/endpoints/debug_files.py
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