Skip to content

fix(mcp): Warn instead of reporting issues on stacktrace-link 5xx - #2131

Open
sentry[bot] wants to merge 1 commit into
mainfrom
seer/fix-code-location-500-log-issue
Open

sentry[bot] wants to merge 1 commit into
mainfrom
seer/fix-code-location-500-log-issue

Conversation

@sentry

@sentry sentry Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

In get_issue_details, resolving the code location via Sentry's /projects/{org}/{project}/stacktrace-link/ endpoint is an optional enrichment. When Sentry returns an HTTP 5xx (e.g. on Python projects), resolveCodeLocation threw an ApiServerError that slipped through the catch block's suppression checks (which only excluded ApiClientError and ConfigurationError) and got sent to Sentry via logIssue.

This change downgrades upstream ApiServerError failures in resolveCodeLocation to logWarn with the status, organization, project, and group ID attached, while continuing to return undefined so that get_issue_details succeeds without the optional Code Location section. Unexpected non-API errors (e.g. bugs or parsing errors) continue to be reported via logIssue.

Fixes MCP-SERVER-G3V

@sentry <feedback>: Autofix iterates on these changes
@sentry stop iterating: Autofix stops iterating on this run

This PR was automatically generated by Sentry. You can adjust this setting at any time.

@vercel

vercel Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
sentry-local Skipped Skipped Nov 9, 2026 10:47pm UTC

Request Review

@sentry
sentry Bot deployed to Actions November 9, 2026 22:47 Active
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Nov 9, 2026

This branch was successfully deployed

1 active and 1 inactive deployments
Actions — cc105c5f Deployed Nov 9, 2026 by sentry[bot] via eval #1758
Preview — cc105c5f Deployed Nov 9, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants