Skip to content

ref(preprod): Remove project query param from artifact URLs#109877

Merged
NicoHinderling merged 2 commits intomasterfrom
remove-project-from-snapshots-url
Mar 4, 2026
Merged

ref(preprod): Remove project query param from artifact URLs#109877
NicoHinderling merged 2 commits intomasterfrom
remove-project-from-snapshots-url

Conversation

@NicoHinderling
Copy link
Contributor

Remove the ?project= query parameter from preprod artifact and comparison URLs.

The project ID is unnecessary in the URL since the artifact ID alone is sufficient to identify the resource. This produces cleaner URLs like https://sentry.sentry.io/preprod/snapshots/70509 instead of https://sentry.sentry.io/preprod/snapshots/70509?project=4509968066281472.

The project ID is not needed in preprod artifact URLs since the
artifact ID is sufficient to identify the resource.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 4, 2026
@NicoHinderling NicoHinderling marked this pull request as ready for review March 4, 2026 16:41
@NicoHinderling NicoHinderling requested a review from a team as a code owner March 4, 2026 16:41
Copy link
Contributor

@chromy chromy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

…URLs

Tests in test_organization_preprod_artifact_assemble.py and
test_templates.py still expected ?project= in generated preprod
artifact URLs after the query parameter was removed from url_utils.py.

Co-Authored-By: Claude <noreply@anthropic.com>
@NicoHinderling NicoHinderling enabled auto-merge (squash) March 4, 2026 16:50
@NicoHinderling NicoHinderling merged commit 3e44ee4 into master Mar 4, 2026
76 checks passed
@NicoHinderling NicoHinderling deleted the remove-project-from-snapshots-url branch March 4, 2026 17:16
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