Skip to content

Conversation

@w0rk3r
Copy link
Contributor

@w0rk3r w0rk3r commented Nov 9, 2025

Proposed commit message

Remove the `script_block_signature` field to improve pipeline performance.

Summary

It seems that the gsub processor I added in #15834 to handle the way PowerShell scripts contain the signature is too expensive and it is causing performance problems.

As I mentioned in #15834, the field was not being populated in my cluster and detection rules telemetry, and will not be populated in most PowerShell scripts as it ends up being split in two different events, so I propose to remove it to solve the problem.

Another point is that most PowerShell scripts with signatures end up having them truncated, since PowerShell logs have size limits, the signature often gets split across two events (Example).

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

@w0rk3r w0rk3r requested a review from marc-gr November 9, 2025 21:20
@w0rk3r w0rk3r self-assigned this Nov 9, 2025
@w0rk3r w0rk3r added bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Nov 9, 2025
@w0rk3r w0rk3r requested review from a team as code owners November 9, 2025 21:20
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @w0rk3r

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

Labels

bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants