ref(flags): Remove organizations:ingest-through-trusted-relays-only#115682
Conversation
github-actions
Bot
added
Scope: Frontend
|
🚨 Warning: This pull request contains Frontend and Backend changes! It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently. Have questions? Please ask in the |
📊 Type Coverage Diff✅ No new type safety issues introduced. Coverage: 93.56% |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
wedamija
left a comment
wedamija
left a comment
There was a problem hiding this comment.
I'm not totally sure if this is fine to remove, would appreciate if someone from @getsentry/ingest could take a look. If we want to keep it, I can move it to be a permanent flag instead
wedamija
force-pushed
the
flag-cleanup/organizations-ingest-through-trusted-relays-only
branch
from
b6fa5ed to
3b7413f
Compare
wedamija
force-pushed
the
flag-cleanup/organizations-ingest-through-trusted-relays-only
branch
from
3b7413f to
a496b65
Compare
wedamija
force-pushed
the
flag-cleanup/organizations-ingest-through-trusted-relays-only
branch
from
a496b65 to
2ff833c
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 3 total unresolved issues (including 2 from previous reviews).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 2ff833c. Configure here.
Dav1dde
left a comment
Dav1dde
left a comment
There was a problem hiding this comment.
This was built as a customer was requesting it, but it is useful in general to all users of Relay. Due to staffing and other things, this was pretty much finished just never rolled out (honestly I thought it was already rolled out).
We should keep this around for now.
wedamija
force-pushed
the
flag-cleanup/organizations-ingest-through-trusted-relays-only
branch
from
2ff833c to
fc84b60
Compare
wedamija
force-pushed
the
flag-cleanup/organizations-ingest-through-trusted-relays-only
branch
from
fc84b60 to
2ff833c
Compare
@Dav1dde Ok, should I make this a permanent feature flag, or just set the flag scanner to ignore it for 6 months or so? |
|
@wedamija if you have the cycles to make it permanent, that'd be amazing |
…permanent Move from temporary.py to permanent.py's flagpole section. The flag remains controllable via flagpole YAML config with api_expose=True, but is now classified as a permanent feature. No behavior change. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
wedamija
force-pushed
the
flag-cleanup/organizations-ingest-through-trusted-relays-only
branch
from
2ff833c to
0499292
Compare
wedamija
deleted the
flag-cleanup/organizations-ingest-through-trusted-relays-only
branch
…115682) Dev-only flag registered Jun 2025, never rolled out to customers. Remove the flag registration and the unreleased gated code paths.
2 participants
Dev-only flag registered Jun 2025, never rolled out to customers. Remove the flag registration and the unreleased gated code paths.