-
Notifications
You must be signed in to change notification settings - Fork 559
Fix versions for changes in required_fileds #4640
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Rule: Tuning - GuidelinesThese guidelines serve as a reminder set of considerations when tuning an existing rule. Documentation and Context
Rule Metadata Checks
Testing and Validation
|
⛔️ Test failed Results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the versioning and ECS field descriptions for several Windows detection rules. Key changes include:
- Updated the updated_date to "2025/04/23".
- Set min_stack_version to "9.0.0" and updated min_stack_comments for accurate ECS field type changes.
- Changes span four rule files related to Windows persistence, group modification, script execution, and security logs.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
File | Description |
---|---|
rules/windows/persistence_user_account_added_to_privileged_group_ad.toml | Updated version and ECS message for winlog.api field type change. |
rules/windows/persistence_group_modification_by_system.toml | Updated version and ECS message for winlog.api field type change. |
rules/windows/execution_windows_script_from_internet.toml | Updated version and ECS message for file.origin_referrer_url and file.origin_url type change. |
rules/windows/defense_evasion_clearing_windows_security_logs.toml | Updated version and ECS message for winlog.api field type change. |
rules/windows/persistence_user_account_added_to_privileged_group_ad.toml
Outdated
Show resolved
Hide resolved
rules/windows/defense_evasion_clearing_windows_security_logs.toml
Outdated
Show resolved
Hide resolved
⛔️ Test failed Results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the minimum stack version and related comment metadata for several Windows detection rules to reflect required changes in ECS field types between stack versions 8.18 and 9.0.0.
- Updated the min_stack_version and min_stack_comments fields for rules with required field changes in winlog.api.
- Updated the min_stack_version and min_stack_comments fields for the rule updating file event fields (file.origin_referrer_url and file.origin_url).
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
rules/windows/persistence_user_account_added_to_privileged_group_ad.toml | Updated metadata with new stack version and comments for winlog.api |
rules/windows/persistence_group_modification_by_system.toml | Updated metadata with new stack version and comments for winlog.api |
rules/windows/execution_windows_script_from_internet.toml | Updated metadata with new stack version and comments for file fields |
rules/windows/defense_evasion_clearing_windows_security_logs.toml | Updated metadata with new stack version and comments for winlog.api |
⛔️ Test failed Results
|
Debugging the "unknown" type for filed Windows Schema Versions
Picking Latest
Finding "winlog.api" in the schema in all possible datastreams ( the filed is not availble in perfmon and service datastreams)
Now Drilling Down on 2.5.1 version
Now Drilling Down on 2.5.0 version
For 9.0.0 The windows integration supported is on ^2.5.0 in all these locally the schemas everything is mapped to keyword and nothing is "unknown" System Schema Versions
The field winlog.api is available only in application, security, system so lets analyse those keys and 1.64.0
1.64.1
1.65.0
1.66.0
1.66.1
1.67.0 , 1.67.1, 1.67.2, 1.67.3
In all these versions locally the schemas everything is mapped to keyword and nothing is "unknown" In the latest beats , |
51359a3
to
be4f23b
Compare
⛔️ Test failed Results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR fixes the stack version and associated comments for the Windows rule to align with updated ECS field types in version 9.0.0.
- Updated the min_stack_version from 8.15.0 to 9.0.0
- Changed the min_stack_comments to reflect the required changes for ECS fields
- Updated the updated_date to the current version's date
Co-authored-by: Copilot <[email protected]>
⛔️ Test failed Results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the minimum stack version and associated comments for the "Execution of a Downloaded Windows Script" rule to reflect changes in required ECS fields for version 9.0.0.
- Updated the min_stack_version, updated_date, and min_stack_comments values in the rule configuration file.
Co-authored-by: Copilot <[email protected]>
⛔️ Test failed Results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the rule metadata to align with changes in required ECS fields.
- Updated min_stack_version from "8.15.0" to "9.0.0"
- Revised updated_date and min_stack_comments to highlight the ECS field type change for file.origin_referrer_url and file.origin_url
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Jonhnathan <[email protected]>
⛔️ Test failed Results
|
⛔️ Test failed Results
|
Pull Request
Issue link(s): Identified while testing #4621
Summary - What I changed
winlog.api filed investigations
For the below rules

- Windows Event Logs Cleared
- User Added to Privileged Group
- Active Directory Group Modification by SYSTEM
We have changes in required filed winlog.api
For the below rule
How To Test
Checklist
bug
,enhancement
,schema
,maintenance
,Rule: New
,Rule: Deprecation
,Rule: Tuning
,Hunt: New
, orHunt: Tuning
so guidelines can be generatedmeta:rapid-merge
label if planning to merge within 24 hoursContributor checklist