feat(types): Add organization ID parsing for DSNs#1202
Open
szokeasaurusrex wants to merge 1 commit into
Open
Conversation
|
szokeasaurusrex
force-pushed
the
szokeasaurusrex/org-id-dsn-parse
branch
from
June 25, 2026 13:08
cbde4ff to
53bd774
Compare
szokeasaurusrex
marked this pull request as ready for review
June 25, 2026 13:10
lcian
removed their request for review
June 25, 2026 14:02
szokeasaurusrex
force-pushed
the
szokeasaurusrex/org-id-dsn-parse
branch
from
July 7, 2026 11:05
53bd774 to
d8f38f3
Compare
szokeasaurusrex
force-pushed
the
szokeasaurusrex/org-id-dsn-parse
branch
from
July 8, 2026 13:56
d8f38f3 to
08f1a80
Compare
szokeasaurusrex
changed the base branch from
master
to
szokeasaurusrex/sampling-strat
July 8, 2026 13:56
szokeasaurusrex
force-pushed
the
szokeasaurusrex/org-id-dsn-parse
branch
from
July 8, 2026 14:04
08f1a80 to
cb59fdf
Compare
giortzisg
approved these changes
Jul 9, 2026
szokeasaurusrex
requested review from
Litarnus and
giortzisg
and removed request for
alexander-alderman-webb
July 20, 2026 15:07
szokeasaurusrex
force-pushed
the
szokeasaurusrex/sampling-strat
branch
from
July 20, 2026 15:52
a32eb15 to
ce74a79
Compare
szokeasaurusrex
force-pushed
the
szokeasaurusrex/org-id-dsn-parse
branch
from
July 20, 2026 15:53
cb59fdf to
28f70b6
Compare
szokeasaurusrex
force-pushed
the
szokeasaurusrex/sampling-strat
branch
from
July 20, 2026 16:23
ce74a79 to
221c434
Compare
szokeasaurusrex
force-pushed
the
szokeasaurusrex/org-id-dsn-parse
branch
2 times, most recently
from
July 20, 2026 16:29
f8efddc to
b72fb2d
Compare
szokeasaurusrex
force-pushed
the
szokeasaurusrex/sampling-strat
branch
from
July 20, 2026 16:29
221c434 to
17bead9
Compare
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit b72fb2d. Configure here.
| #[test] | ||
| fn test_dsn_org_id_rejects_empty_first_label() { | ||
| assert_org_id("https://username@.ingest.sentry.io/42", None); | ||
| } |
There was a problem hiding this comment.
Broken empty-label org ID test
Medium Severity
test_dsn_org_id_rejects_empty_first_label builds a DSN from https://username@.ingest.sentry.io/42. That host has an empty DNS label, which the url crate rejects per the URL Standard, so Dsn::from_str(...).unwrap() panics before org_id() is exercised. The case is unreachable through a valid Dsn.
Reviewed by Cursor Bugbot for commit b72fb2d. Configure here.
Litarnus
approved these changes
Jul 21, 2026
| /// - `o{orgid}.ingest.sentry.io` | ||
| fn extract_org_id(host: &str) -> Option<&str> { | ||
| // First split on the dots, up to 6 segments. | ||
| let mut host_iter = host.splitn(6, '.'); |
There was a problem hiding this comment.
Just curious, are you splitting it into 6 segments to make sure the last one is always None?
| None, | ||
| ); | ||
| } | ||
|
|
There was a problem hiding this comment.
You might want to add a test for a hostname with a 6th part that is not None, e.g. https://username@o123.ingest.de.sentry.io.com/42
giortzisg
approved these changes
Jul 22, 2026
Expose `Dsn::org_id` so callers can derive the SDK organization ID needed for strict trace continuation. Parse only Sentry SaaS ingest DSN hosts, including regional hosts, and return `None` for self-hosted, Relay, malformed, and overflow cases. Closes [#1159](#1159) Closes [RUST-233](https://linear.app/getsentry/issue/RUST-233)
szokeasaurusrex
force-pushed
the
szokeasaurusrex/org-id-dsn-parse
branch
from
July 24, 2026 10:11
b72fb2d to
a53202a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Expose
Dsn::org_idso callers can derive the SDK organization ID needed for strict trace continuation. Parse only Sentry SaaS ingest DSN hosts, including regional hosts, and returnNonefor self-hosted, Relay, malformed, and overflow cases.Closes #1159
Closes RUST-233