Skip to content

[Tuning] Remote Management Access Launch After MSI Install - #6767

Merged
shashank-elastic merged 2 commits into
mainfrom
Samirbous-patch-8
Sep 15, 2026
Merged

shashank-elastic merged 2 commits into
mainfrom
Samirbous-patch-8

Conversation

@Samirbous

Copy link
Copy Markdown
Contributor

Resolve an FN - adds powershell and cmd as parent of msiexec.

@Samirbous Samirbous self-assigned this Sep 11, 2026
Copilot AI lite review requested due to automatic review settings September 11, 2026 08:35
@Samirbous Samirbous added the Rule: Tuning tweaking or tuning an existing rule label Sep 11, 2026
@botelastic botelastic Bot added Domain: Endpoint OS: Windows windows related rules labels Sep 11, 2026
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Sep 11, 2026

Copy link
Copy Markdown

⛔️ Test failed

Results
  • ❌ Remote Management Access Launch After MSI Install (eql)
    • coverage_issue: no_rta
    • stack_validation_failed: no_rta

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Add pwsh.exe, renamed msiexec.exe coverage, and the required taxonomy tags.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates Windows EQL detection for RMM launches following MSI installation.

Changes:

  • Adds PowerShell and Command Prompt as possible msiexec.exe parents.
  • Updates the rule timestamp and parent-process matching.
File summaries
File Description
rules/windows/command_and_control_rmm_after_msi_install.toml Expands MSI parent-process detection.
Review details

Suppressed comments (1)

rules/windows/command_and_control_rmm_after_msi_install.toml:5

  • The rule metadata still lacks the required Platform: Windows and Rule Type: Event Correlation (EQL) taxonomy tags. Add both to the existing tags array while updating this rule.
updated_date = "2026/09/11"
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines 90 to +91
[process where host.os.type == "windows" and event.type == "start" and process.name : "msiexec.exe" and
process.args : ("/i*", "-i*") and process.parent.name : ("explorer.exe", "sihost.exe")]
process.args : ("/i*", "-i*") and process.parent.name : ("explorer.exe", "sihost.exe", "powershell.exe", "cmd.exe")]
@elastic-vault-github-plugin-prod

elastic-vault-github-plugin-prod Bot commented Sep 14, 2026

Copy link
Copy Markdown

⛔️ Test failed

Results
  • ❌ Remote Management Access Launch After MSI Install (eql)
    • coverage_issue: no_rta
    • stack_validation_failed: no_rta

@shashank-elastic
shashank-elastic added this pull request to the merge queue Sep 15, 2026
Merged via the queue into main with commit 37936a4 Sep 15, 2026
10 checks passed
@shashank-elastic
shashank-elastic deleted the Samirbous-patch-8 branch September 15, 2026 03:27
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

Development

Successfully merging this pull request may close these issues.

5 participants