-
Notifications
You must be signed in to change notification settings - Fork 78
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
[FEATURE] SigmaRule Parser - Remove replacing "." to "_" in field names #375
Comments
@eirsep and @petardz Just wanted to confirm that there are aliases that we see on the detector mapping screen. So, as per the configured rules, we provide mapping for the aliases against our field present in the index. I have tried with the sigma rule while configuring the cloudtrail detector, so at that time I was not able to map the field "userIdentity.type", I see there is alias called aws-cloudtrail-user_identity-type, but I don't see any mappings for the same in security analytics repository. So, I guess this is the issue. Also, I'm able to generate findings for all the flattened data with the sigma rule configuration. But, somehow, I am not able to generate findings/alerts for the nested json data (which is parsed and ingested as per the ECS format). So can anyone help me understand what I might be missing? Just to share context, I am using Security Analytics Plugin from the OpenSearch version 2.6x (latest), which is running on docker image. For detector configuration, I have used cloudtrail detector with all 31 out-of-the-box sigma rules available. References: |
You're right, mappings are buggy in 2.6. There are also other examples where this is a case. You can work around this temporary, by defining alias mappings in your index manually:
This issue is in Alerting and it is addressed by this PR: link |
Fixed in 2.9 |
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
Is your feature request related to a problem?
During first release of SAP we had issues with fields containing "." in sigma rules. That issue was fixed since then and we don't have a need for replacing "." char in field names anymore.
What solution would you like?
Remove replacing "." characters.
The text was updated successfully, but these errors were encountered: