Skip to content

Add EmailEvent filtering validator support - #15412

Open
adbhogeGit wants to merge 1 commit into
masterfrom
adbhoge-microsoft-emailevent-filtering-ci
Open

adbhogeGit wants to merge 1 commit into
masterfrom
adbhoge-microsoft-emailevent-filtering-ci

Conversation

@adbhogeGit

Copy link
Copy Markdown
Contributor

Fixes #

Proposed Changes

  • Register all standard EmailEvent filtering parameters in both ASIM CI validators.
  • Validate recipient matching across EmailToRecipients, EmailCCRecipients, and EmailBCCRecipients; test sender/subject has-any, direction/location membership, IP prefixes, event result, defaults, and pack behavior.
  • Use the mapped EventStartTime/EventEndTime columns for datetime filtering and check in-range and out-of-range boundaries.
  • For sparse, constant, or unavailable fields, keep match/default/absent checks and report non-selective coverage as partial instead of failing solely for low cardinality.
  • Add deterministic regression tests and document the legacy validator behavior.

Required items, please complete

Change(s):

  • Added EmailEvent support to .script/tests/asimParsersTest/ASimFilteringTest.py and ASIM/dev/ASimTester/filteringTest/ASimFilteringTest.py.
  • Added focused regression tests for the EmailEvent catalog, dynamic recipient arrays, Message-like sender/subject data, sparse MailLog-like data, datetime boundaries, unavailable columns, pack/disabled controls, and existing schemas.

Reason for Change(s):

Version Updated:

  • N/A — this change does not update a detection or analytic rule template.

Testing Completed:

  • & 'C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe' -m unittest discover -s .script\tests\asimParsersTest -p 'test_asim_filtering_validator.py' -v — passed (12 tests).
  • & 'C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe' -m py_compile .script\tests\asimParsersTest\ASimFilteringTest.py .script\tests\asimParsersTest\test_asim_filtering_validator.py ASIM\dev\ASimTester\filteringTest\ASimFilteringTest.py — passed.
  • git --no-pager diff --check — passed.
  • No live Azure workspace query or hosted parser-filter run was performed.

Checked that the validations are passing and have addressed any issues that are present:

  • Local deterministic regression and syntax checks pass. Live workspace/hosted validation remains to be rerun after merge.

Register standard EmailEvent filters in both CI validators, validate recipient arrays and event-time boundaries, and report sparse data as partial while still checking matches and absent values.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 9d6cfd8c-4385-4bd6-8f53-a9dc74446ece
@adbhogeGit
adbhogeGit requested review from a team as code owners October 23, 2026 19:29
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