Skip to content

fix(workflows): Validate delayed workflow tag filter keys#117344

Merged
kcons merged 1 commit into
masterfrom
kcons/dontnone
Jun 10, 2026
Merged

fix(workflows): Validate delayed workflow tag filter keys#117344
kcons merged 1 commit into
masterfrom
kcons/dontnone

Conversation

@kcons

@kcons kcons commented Jun 10, 2026

Copy link
Copy Markdown
Member

Reject invalid persisted tag filter keys with TAG_KEY_RE before building Snuba columns, and return zero results instead of crashing.

Fixes SENTRY-4EBM

@kcons kcons requested review from a team as code owners June 10, 2026 18:26
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 10, 2026
Comment thread src/sentry/api/utils.py Dismissed

@ceorourke ceorourke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, would it make sense to clean up invalid rows? I'm not entirely sure how we'd be able to go about that

@kcons

kcons commented Jun 10, 2026

Copy link
Copy Markdown
Member Author

@ceorourke just to confirm, the security bot flag here isn't something we actually care about, right? Or am I mistaken?

@ceorourke

Copy link
Copy Markdown
Member

@ceorourke just to confirm, the security bot flag here isn't something we actually care about, right? Or am I mistaken?

I think so, afaik it will just go to a sentry issue

@kcons kcons merged commit 8f25236 into master Jun 10, 2026
86 checks passed
@kcons kcons deleted the kcons/dontnone branch June 10, 2026 18:52
amy-chen23 pushed a commit that referenced this pull request Jun 10, 2026
Reject invalid persisted tag filter keys with TAG_KEY_RE before building
Snuba columns, and return zero results instead of crashing.

Fixes SENTRY-4EBM
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.

3 participants