Skip to content

fix(scoping): Pass X-Viewer-Context on supergroups list/get requests to Seer#110773

Merged
gricha merged 1 commit intomasterfrom
greg/fix-supergroups-viewer-context
Mar 16, 2026
Merged

fix(scoping): Pass X-Viewer-Context on supergroups list/get requests to Seer#110773
gricha merged 1 commit intomasterfrom
greg/fix-supergroups-viewer-context

Conversation

@gricha
Copy link
Member

@gricha gricha commented Mar 16, 2026

Summary

  • Adds viewer_context parameter to make_supergroups_list_request and make_supergroups_get_request
  • Passes SeerViewerContext(organization_id, user_id) from the supergroups list and detail endpoints

These endpoints were the only supergroups call sites not sending the X-Viewer-Context header to Seer, causing OrganizationScoped validation errors (SEER-7P5).

Test plan

  • Verified locally: Seer logs ActorContext set from X-Viewer-Context when header is present
  • After deploy, SEER-7P5 event volume should drop to zero

Agent transcript: https://claudescope.sentry.dev/share/-Im9fOrNO5oHm22L3mTaH_pN_j6rK5n_QkFSPygYm5E

@gricha gricha requested a review from a team as a code owner March 16, 2026 18:29
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 16, 2026
…quests to Seer

Add `viewer_context` as a required param on `make_supergroups_list_request`
and `make_supergroups_get_request`, and pass `SeerViewerContext` from both
call sites. These were the only supergroups endpoints not sending the header,
causing OrganizationScoped validation errors on the Seer side (SEER-7P5).
@gricha gricha force-pushed the greg/fix-supergroups-viewer-context branch from 699e73a to 13a6bd6 Compare March 16, 2026 18:34
@gricha gricha merged commit 8b9955a into master Mar 16, 2026
59 checks passed
@gricha gricha deleted the greg/fix-supergroups-viewer-context branch March 16, 2026 19:43
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