Skip to content

feat(seer-infra-telemetry): Add DatadogIdentityProvider for OAuth2#117035

Merged
srest2021 merged 58 commits into
masterfrom
srest2021/CW-1469
Jun 11, 2026
Merged

feat(seer-infra-telemetry): Add DatadogIdentityProvider for OAuth2#117035
srest2021 merged 58 commits into
masterfrom
srest2021/CW-1469

raise identitynotvalid if no dcr credentials; fix tests

4cdd631
Select commit
Loading
Failed to load commit list.
@sentry/warden / warden: sentry-backend-bugs completed Jun 11, 2026 in 2m 15s

1 issue

sentry-backend-bugs: Found 1 issue (1 medium)

Medium

Unguarded header key access raises KeyError when `mcp-session-id` is absent - `src/sentry/identity/datadog/provider.py:53`

init_resp.headers["mcp-session-id"] will raise an unhandled KeyError if the MCP server returns a 2xx response without that header; wrap this in a guard or use .get() and raise IdentityNotValid on None.

Also found at:

  • src/sentry/identity/datadog/provider.py:313

⏱ 1m 34s · 230.4k in / 10.4k out · $0.52

Annotations

Check warning on line 53 in src/sentry/identity/datadog/provider.py

See this annotation in the file changed.

@sentry-warden sentry-warden / warden: sentry-backend-bugs

Unguarded header key access raises KeyError when `mcp-session-id` is absent

`init_resp.headers["mcp-session-id"]` will raise an unhandled `KeyError` if the MCP server returns a 2xx response without that header; wrap this in a guard or use `.get()` and raise `IdentityNotValid` on `None`.

Check warning on line 313 in src/sentry/identity/datadog/provider.py

See this annotation in the file changed.

@sentry-warden sentry-warden / warden: sentry-backend-bugs

[CQK-A2K] Unguarded header key access raises KeyError when `mcp-session-id` is absent (additional location)

`init_resp.headers["mcp-session-id"]` will raise an unhandled `KeyError` if the MCP server returns a 2xx response without that header; wrap this in a guard or use `.get()` and raise `IdentityNotValid` on `None`.