Skip to content

feat(viewer-context): Switch Seer API to JWT X-Viewer-Context header#112894

Open
gricha wants to merge 3 commits intomasterfrom
gricha/feat/seer-jwt-viewer-context
Open

feat(viewer-context): Switch Seer API to JWT X-Viewer-Context header#112894
gricha wants to merge 3 commits intomasterfrom
gricha/feat/seer-jwt-viewer-context

Conversation

@gricha
Copy link
Copy Markdown
Member

@gricha gricha commented Apr 14, 2026

Replace the legacy two-header format (X-Viewer-Context JSON +
X-Viewer-Context-Signature HMAC) with a single X-Viewer-Context
JWT header in make_signed_seer_api_request().

Seer already accepts JWT via is_jwt_viewer_context / decode_jwt_viewer_context
(getsentry/seer@b22dcb62).

Do not merge until Seer's JWT acceptance is deployed.

Depends on #112765, #112875.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 14, 2026
@gricha gricha marked this pull request as ready for review April 14, 2026 05:18
@gricha gricha requested review from a team as code owners April 14, 2026 05:18
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 d74f591. Configure here.

gricha and others added 3 commits April 14, 2026 13:30
…header

Replace the legacy two-header format (X-Viewer-Context JSON +
X-Viewer-Context-Signature HMAC) with a single X-Viewer-Context
JWT header when calling Seer. Seer already accepts JWT format.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
This function's only caller was replaced by encode_viewer_context in the
JWT migration. No other code imports or calls it.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…utils

The previous commit removed this function but it is still imported by
client_utils.py:get_proxy_headers(). Restore it until that caller is
migrated to the JWT approach.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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