Skip to content
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

[Rule Tuning] Adjust Index Pattern on Windows rules to support WEF #2438

Merged
merged 3 commits into from
Dec 21, 2022

Conversation

w0rk3r
Copy link
Contributor

@w0rk3r w0rk3r commented Dec 21, 2022

Issues

Resolves #2417 & #2415

Summary

When users ingest windows logs via WEF, they are collected by the Windows Integration instead of the System Integration, causing the logs to be written to the logs-windows.* index. This PR adds the index pattern to those rules that are compatible.

@w0rk3r w0rk3r added Rule: Tuning tweaking or tuning an existing rule OS: Windows windows related rules Domain: Endpoint labels Dec 21, 2022
@w0rk3r w0rk3r self-assigned this Dec 21, 2022
@w0rk3r
Copy link
Contributor Author

w0rk3r commented Dec 21, 2022

A detail missed here: #2415 (comment); I'll be reviewing rules that include those and adjust accordingly

@w0rk3r
Copy link
Contributor Author

w0rk3r commented Dec 21, 2022

Instead of host.id, we are using winlog.computer_name, which should be compatible with both integrations. Reference: elastic/beats#13706

Copy link
Contributor

@terrancedejesus terrancedejesus left a comment

Choose a reason for hiding this comment

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

LGTM. The index pattern should still be picked up for build time fields and recommend the Windows integration for either so upstream in Kibana this should be fine for that feature. Reference - #2429

@w0rk3r w0rk3r merged commit 9c1bd50 into main Dec 21, 2022
@w0rk3r w0rk3r deleted the wef_index_pattern branch December 21, 2022 14:30
protectionsmachine pushed a commit that referenced this pull request Dec 21, 2022
…2438)

* [Rule Tuning] Adjust Index Pattern on Windows rules to support WEF

* s/host.id/winlog.computer_name

Removed changes from:
- rules/windows/credential_access_bruteforce_admin_account.toml
- rules/windows/credential_access_bruteforce_multiple_logon_failure_followed_by_success.toml
- rules/windows/credential_access_bruteforce_multiple_logon_failure_same_srcip.toml
- rules/windows/credential_access_dcsync_replication_rights.toml
- rules/windows/credential_access_disable_kerberos_preauth.toml
- rules/windows/credential_access_ldap_attributes.toml
- rules/windows/credential_access_lsass_memdump_handle_access.toml
- rules/windows/credential_access_remote_sam_secretsdump.toml
- rules/windows/credential_access_saved_creds_vault_winlog.toml
- rules/windows/credential_access_seenabledelegationprivilege_assigned_to_user.toml
- rules/windows/credential_access_shadow_credentials.toml
- rules/windows/credential_access_spn_attribute_modified.toml
- rules/windows/credential_access_suspicious_winreg_access_via_sebackup_priv.toml
- rules/windows/defense_evasion_clearing_windows_security_logs.toml
- rules/windows/discovery_privileged_localgroup_membership.toml
- rules/windows/lateral_movement_remote_service_installed_winlog.toml
- rules/windows/lateral_movement_remote_task_creation_winlog.toml
- rules/windows/lateral_movement_service_control_spawned_script_int.toml
- rules/windows/persistence_ad_adminsdholder.toml
- rules/windows/persistence_dontexpirepasswd_account.toml
- rules/windows/persistence_msds_alloweddelegateto_krbtgt.toml
- rules/windows/persistence_remote_password_reset.toml
- rules/windows/persistence_scheduled_task_creation_winlog.toml
- rules/windows/persistence_scheduled_task_updated.toml
- rules/windows/persistence_sdprop_exclusion_dsheuristics.toml
- rules/windows/persistence_service_windows_service_winlog.toml
- rules/windows/persistence_temp_scheduled_task.toml
- rules/windows/persistence_user_account_added_to_privileged_group_ad.toml
- rules/windows/privilege_escalation_create_process_as_different_user.toml
- rules/windows/privilege_escalation_credroaming_ldap.toml
- rules/windows/privilege_escalation_group_policy_iniscript.toml
- rules/windows/privilege_escalation_group_policy_privileged_groups.toml
- rules/windows/privilege_escalation_group_policy_scheduled_task.toml
- rules/windows/privilege_escalation_krbrelayup_service_creation.toml
- rules/windows/privilege_escalation_samaccountname_spoofing_attack.toml
- rules/windows/privilege_escalation_suspicious_dnshostname_update.toml
- rules/windows/privilege_escalation_windows_service_via_unusual_client.toml

(selectively cherry picked from commit 9c1bd50)
protectionsmachine pushed a commit that referenced this pull request Dec 21, 2022
…2438)

* [Rule Tuning] Adjust Index Pattern on Windows rules to support WEF

* s/host.id/winlog.computer_name

Removed changes from:
- rules/windows/credential_access_bruteforce_admin_account.toml
- rules/windows/credential_access_bruteforce_multiple_logon_failure_followed_by_success.toml
- rules/windows/credential_access_bruteforce_multiple_logon_failure_same_srcip.toml
- rules/windows/credential_access_dcsync_replication_rights.toml
- rules/windows/credential_access_disable_kerberos_preauth.toml
- rules/windows/credential_access_ldap_attributes.toml
- rules/windows/credential_access_lsass_memdump_handle_access.toml
- rules/windows/credential_access_remote_sam_secretsdump.toml
- rules/windows/credential_access_saved_creds_vault_winlog.toml
- rules/windows/credential_access_seenabledelegationprivilege_assigned_to_user.toml
- rules/windows/credential_access_shadow_credentials.toml
- rules/windows/credential_access_spn_attribute_modified.toml
- rules/windows/credential_access_suspicious_winreg_access_via_sebackup_priv.toml
- rules/windows/defense_evasion_clearing_windows_security_logs.toml
- rules/windows/discovery_privileged_localgroup_membership.toml
- rules/windows/lateral_movement_remote_service_installed_winlog.toml
- rules/windows/lateral_movement_remote_task_creation_winlog.toml
- rules/windows/lateral_movement_service_control_spawned_script_int.toml
- rules/windows/persistence_ad_adminsdholder.toml
- rules/windows/persistence_dontexpirepasswd_account.toml
- rules/windows/persistence_msds_alloweddelegateto_krbtgt.toml
- rules/windows/persistence_remote_password_reset.toml
- rules/windows/persistence_scheduled_task_creation_winlog.toml
- rules/windows/persistence_scheduled_task_updated.toml
- rules/windows/persistence_sdprop_exclusion_dsheuristics.toml
- rules/windows/persistence_service_windows_service_winlog.toml
- rules/windows/persistence_temp_scheduled_task.toml
- rules/windows/persistence_user_account_added_to_privileged_group_ad.toml
- rules/windows/privilege_escalation_create_process_as_different_user.toml
- rules/windows/privilege_escalation_credroaming_ldap.toml
- rules/windows/privilege_escalation_group_policy_iniscript.toml
- rules/windows/privilege_escalation_group_policy_privileged_groups.toml
- rules/windows/privilege_escalation_group_policy_scheduled_task.toml
- rules/windows/privilege_escalation_krbrelayup_service_creation.toml
- rules/windows/privilege_escalation_samaccountname_spoofing_attack.toml
- rules/windows/privilege_escalation_suspicious_dnshostname_update.toml
- rules/windows/privilege_escalation_windows_service_via_unusual_client.toml

(selectively cherry picked from commit 9c1bd50)
protectionsmachine pushed a commit that referenced this pull request Dec 21, 2022
…2438)

* [Rule Tuning] Adjust Index Pattern on Windows rules to support WEF

* s/host.id/winlog.computer_name

Removed changes from:
- rules/windows/credential_access_bruteforce_admin_account.toml
- rules/windows/credential_access_bruteforce_multiple_logon_failure_followed_by_success.toml
- rules/windows/credential_access_bruteforce_multiple_logon_failure_same_srcip.toml
- rules/windows/credential_access_dcsync_replication_rights.toml
- rules/windows/credential_access_disable_kerberos_preauth.toml
- rules/windows/credential_access_ldap_attributes.toml
- rules/windows/credential_access_lsass_memdump_handle_access.toml
- rules/windows/credential_access_remote_sam_secretsdump.toml
- rules/windows/credential_access_saved_creds_vault_winlog.toml
- rules/windows/credential_access_seenabledelegationprivilege_assigned_to_user.toml
- rules/windows/credential_access_shadow_credentials.toml
- rules/windows/credential_access_spn_attribute_modified.toml
- rules/windows/credential_access_suspicious_winreg_access_via_sebackup_priv.toml
- rules/windows/defense_evasion_clearing_windows_security_logs.toml
- rules/windows/discovery_privileged_localgroup_membership.toml
- rules/windows/lateral_movement_remote_service_installed_winlog.toml
- rules/windows/lateral_movement_remote_task_creation_winlog.toml
- rules/windows/lateral_movement_service_control_spawned_script_int.toml
- rules/windows/persistence_ad_adminsdholder.toml
- rules/windows/persistence_dontexpirepasswd_account.toml
- rules/windows/persistence_msds_alloweddelegateto_krbtgt.toml
- rules/windows/persistence_remote_password_reset.toml
- rules/windows/persistence_scheduled_task_creation_winlog.toml
- rules/windows/persistence_scheduled_task_updated.toml
- rules/windows/persistence_sdprop_exclusion_dsheuristics.toml
- rules/windows/persistence_service_windows_service_winlog.toml
- rules/windows/persistence_temp_scheduled_task.toml
- rules/windows/persistence_user_account_added_to_privileged_group_ad.toml
- rules/windows/privilege_escalation_create_process_as_different_user.toml
- rules/windows/privilege_escalation_credroaming_ldap.toml
- rules/windows/privilege_escalation_group_policy_iniscript.toml
- rules/windows/privilege_escalation_group_policy_privileged_groups.toml
- rules/windows/privilege_escalation_group_policy_scheduled_task.toml
- rules/windows/privilege_escalation_krbrelayup_service_creation.toml
- rules/windows/privilege_escalation_samaccountname_spoofing_attack.toml
- rules/windows/privilege_escalation_suspicious_dnshostname_update.toml
- rules/windows/privilege_escalation_windows_service_via_unusual_client.toml

(selectively cherry picked from commit 9c1bd50)
protectionsmachine pushed a commit that referenced this pull request Dec 21, 2022
…2438)

* [Rule Tuning] Adjust Index Pattern on Windows rules to support WEF

* s/host.id/winlog.computer_name

Removed changes from:
- rules/windows/credential_access_bruteforce_admin_account.toml
- rules/windows/credential_access_bruteforce_multiple_logon_failure_followed_by_success.toml
- rules/windows/credential_access_bruteforce_multiple_logon_failure_same_srcip.toml
- rules/windows/credential_access_dcsync_replication_rights.toml
- rules/windows/credential_access_disable_kerberos_preauth.toml
- rules/windows/credential_access_ldap_attributes.toml
- rules/windows/credential_access_lsass_memdump_handle_access.toml
- rules/windows/credential_access_remote_sam_secretsdump.toml
- rules/windows/credential_access_saved_creds_vault_winlog.toml
- rules/windows/credential_access_seenabledelegationprivilege_assigned_to_user.toml
- rules/windows/credential_access_shadow_credentials.toml
- rules/windows/credential_access_spn_attribute_modified.toml
- rules/windows/credential_access_suspicious_winreg_access_via_sebackup_priv.toml
- rules/windows/defense_evasion_clearing_windows_security_logs.toml
- rules/windows/discovery_privileged_localgroup_membership.toml
- rules/windows/lateral_movement_remote_service_installed_winlog.toml
- rules/windows/lateral_movement_remote_task_creation_winlog.toml
- rules/windows/lateral_movement_service_control_spawned_script_int.toml
- rules/windows/persistence_ad_adminsdholder.toml
- rules/windows/persistence_dontexpirepasswd_account.toml
- rules/windows/persistence_msds_alloweddelegateto_krbtgt.toml
- rules/windows/persistence_remote_password_reset.toml
- rules/windows/persistence_scheduled_task_creation_winlog.toml
- rules/windows/persistence_scheduled_task_updated.toml
- rules/windows/persistence_sdprop_exclusion_dsheuristics.toml
- rules/windows/persistence_service_windows_service_winlog.toml
- rules/windows/persistence_temp_scheduled_task.toml
- rules/windows/persistence_user_account_added_to_privileged_group_ad.toml
- rules/windows/privilege_escalation_create_process_as_different_user.toml
- rules/windows/privilege_escalation_credroaming_ldap.toml
- rules/windows/privilege_escalation_group_policy_iniscript.toml
- rules/windows/privilege_escalation_group_policy_privileged_groups.toml
- rules/windows/privilege_escalation_group_policy_scheduled_task.toml
- rules/windows/privilege_escalation_krbrelayup_service_creation.toml
- rules/windows/privilege_escalation_samaccountname_spoofing_attack.toml
- rules/windows/privilege_escalation_suspicious_dnshostname_update.toml
- rules/windows/privilege_escalation_windows_service_via_unusual_client.toml

(selectively cherry picked from commit 9c1bd50)
protectionsmachine pushed a commit that referenced this pull request Dec 21, 2022
…2438)

* [Rule Tuning] Adjust Index Pattern on Windows rules to support WEF

* s/host.id/winlog.computer_name

(cherry picked from commit 9c1bd50)
protectionsmachine pushed a commit that referenced this pull request Dec 21, 2022
…2438)

* [Rule Tuning] Adjust Index Pattern on Windows rules to support WEF

* s/host.id/winlog.computer_name

(cherry picked from commit 9c1bd50)
protectionsmachine pushed a commit that referenced this pull request Dec 21, 2022
…2438)

* [Rule Tuning] Adjust Index Pattern on Windows rules to support WEF

* s/host.id/winlog.computer_name

(cherry picked from commit 9c1bd50)
protectionsmachine pushed a commit that referenced this pull request Dec 21, 2022
…2438)

* [Rule Tuning] Adjust Index Pattern on Windows rules to support WEF

* s/host.id/winlog.computer_name

(cherry picked from commit 9c1bd50)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport: auto Domain: Endpoint OS: Windows windows related rules Rule: Tuning tweaking or tuning an existing rule
Projects
None yet
3 participants