Skip to content

fix(filters): normalize actionResult to canonical values#2366

Open
JocLRojas wants to merge 1 commit into
v11from
backlog/normalize-filter-action-results
Open

fix(filters): normalize actionResult to canonical values#2366
JocLRojas wants to merge 1 commit into
v11from
backlog/normalize-filter-action-results

Conversation

@JocLRojas

Copy link
Copy Markdown
Contributor

Detailed explanation of the changes

Normalize the actionResult field across seven pipeline filters so downstream rules, dashboards, and correlations receive a consistent, canonical vocabulary (denied / blocked / failed / success / accepted) instead of vendor-specific strings.

Reasoning behind these changes

Rules, dashboards and correlations across UTMStack depend on actionResult being one of a small, well-known set of values. Vendor logs use inconsistent casing and vocabulary (Denied, denied, Blocked, Failed, Failure, Succeeded, Success, etc.), which was leaking straight into actionResult through several rename steps. That leak broke downstream matching whenever a vendor changed casing or wording.

This change:

  1. Prevents rename steps from overwriting actionResult with non-canonical values, using a negative regexMatch guard.
  2. Explicitly maps the recognized signals to canonical values (denied, blocked, failed, success, accepted) via add steps.

The result is that correlation rules that filter by actionResult behave predictably regardless of the source vendor's spelling or casing.

Issue

No related issue.

…ked/failed/success) across ESET, Azure, Meraki, GCP, Linux, O365 and Sophos
@JocLRojas
JocLRojas requested a review from a team July 21, 2026 18:38
@github-actions

Copy link
Copy Markdown

✅ AI review — Approved

No issues detected in this diff.

architecture (gemini-3-flash-lite) — clean

Summary: No reviewable changes in this diff (excluded paths only).

No findings.

bugs (gemini-3-flash-lite) — clean

Summary: No reviewable changes in this diff (excluded paths only).

No findings.

security (gemini-3-flash-lite) — clean

Summary: No reviewable changes in this diff (excluded paths only).

No findings.

@utmstackprapprover utmstackprapprover Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved — no blocking issues, deps OK, authorized author. Any non-blocking warnings are listed above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant