Skip to content

fix(issues): Handle has:trace on error events - #121271

Open
scttcper wants to merge 4 commits into
masterfrom
scttcper/has-trace-filter
Open

fix(issues): Handle has:trace on error events#121271
scttcper wants to merge 4 commits into
masterfrom
scttcper/has-trace-filter

Conversation

@scttcper

@scttcper scttcper commented Aug 4, 2026

Copy link
Copy Markdown
Member

The has:trace and has:trace.span filters on error event stats were comparing promoted trace values against an empty string, returning a 400 instead of chart data.

example

The has:trace filter compared promoted UUID trace IDs against an empty string, so error event stats returned a 400. Check context key presence instead and cover the issue-scoped event and user series.\n\nCo-Authored-By: Codex <noreply@openai.com>
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 4, 2026
scttcper and others added 3 commits August 4, 2026 16:51
Spell out the behavior this endpoint test protects.

Co-Authored-By: Codex <noreply@openai.com>
The span filter hit the same empty-value conversion problem as trace IDs, but failed as an invalid hex integer. Use context key presence for both and cover their filtering behavior.

Co-Authored-By: Codex <noreply@openai.com>
Use one setup and exercise both trace existence filters in the same behavior test.

Co-Authored-By: Codex <noreply@openai.com>
@scttcper scttcper changed the title fix(search): Handle has:trace on error events fix(issues): Handle has:trace on error events Aug 4, 2026
@scttcper
scttcper marked this pull request as ready for review August 5, 2026 00:09
@scttcper
scttcper requested a review from a team as a code owner August 5, 2026 00:09
@scttcper
scttcper requested a review from a team August 5, 2026 00:09
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.

1 participant