Skip to content

feat: add SENTRY_ALLOWED_IPS to allow IP, overwrite SENTRY_DISALLOWED…#115773

Merged
markstory merged 2 commits into
getsentry:masterfrom
fe80:feat/add-allowed-ips
May 22, 2026
Merged

feat: add SENTRY_ALLOWED_IPS to allow IP, overwrite SENTRY_DISALLOWED…#115773
markstory merged 2 commits into
getsentry:masterfrom
fe80:feat/add-allowed-ips

Conversation

@fe80
Copy link
Copy Markdown
Contributor

@fe80 fe80 commented May 19, 2026

…_IPS

This feature add the possibility to overwrite SENTRY_DISALLOWED with a simple IP (like 10.1.2.31 inside the default blocked range 10.0.0.0/8`)

This PR solve #100943

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@fe80 fe80 requested review from a team as code owners May 19, 2026 07:41
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 19, 2026
@fe80 fe80 force-pushed the feat/add-allowed-ips branch from f1065a2 to 15b3486 Compare May 19, 2026 07:43
Comment thread src/sentry/net/socket.py
Comment thread tests/sentry/net/test_socket.py
Comment thread src/sentry/testutils/helpers/socket.py
@fe80 fe80 force-pushed the feat/add-allowed-ips branch 2 times, most recently from 678fd6d to 3d2165a Compare May 20, 2026 07:04
…_IPS

Signed-off-by: fe80 <steffyfort@gmail.com>
@fe80 fe80 force-pushed the feat/add-allowed-ips branch from 3d2165a to 6cd4c31 Compare May 20, 2026 08:35
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6cd4c31. Configure here.

Comment thread src/sentry/net/socket.py
@markstory markstory added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label May 20, 2026
@markstory markstory self-assigned this May 21, 2026
@github-actions github-actions Bot removed the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label May 21, 2026
@markstory markstory added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label May 22, 2026
@markstory markstory merged commit c9ada5f into getsentry:master May 22, 2026
142 of 146 checks passed
@markstory
Copy link
Copy Markdown
Member

Thank you 🎉

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 Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants