Skip to content

fix(performance): Rename HTTP method attribute for EAP txn summary profiles#110771

Merged
mjq merged 1 commit intomasterfrom
mjq/txn-summary-eap-profiles
Mar 16, 2026
Merged

fix(performance): Rename HTTP method attribute for EAP txn summary profiles#110771
mjq merged 1 commit intomasterfrom
mjq/txn-summary-eap-profiles

Conversation

@mjq
Copy link
Copy Markdown
Member

@mjq mjq commented Mar 16, 2026

EAP spans use http.method instead of request.method for the HTTP request method field. The Insights landing pages add request.method to the query, so rename it before passing to the profiles query when EAP is enabled.

EAP spans use `http.method` instead of `request.method` for the HTTP
request method field. The Insights landing pages add `request.method`
to the query, so rename it before passing to the profiles query when
EAP is enabled. This matches the pattern already used in relatedIssues.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@mjq
Copy link
Copy Markdown
Member Author

mjq commented Mar 16, 2026

@sentry review

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 16, 2026
@mjq mjq marked this pull request as ready for review March 16, 2026 18:23
@mjq mjq requested a review from a team as a code owner March 16, 2026 18:23
@mjq mjq changed the title fix(profiling): Rename request.method to http.method for EAP profiles fix(profiling): Rename HTTP method attribute for EAP txn summary profiles Mar 16, 2026
@mjq mjq changed the title fix(profiling): Rename HTTP method attribute for EAP txn summary profiles fix(performance): Rename HTTP method attribute for EAP txn summary profiles Mar 16, 2026
@mjq mjq merged commit 6ae6177 into master Mar 16, 2026
68 checks passed
@mjq mjq deleted the mjq/txn-summary-eap-profiles branch March 16, 2026 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants