Skip to content

Conversation

@dan21san
Copy link
Contributor

@dan21san dan21san commented Jul 2, 2024

Summary of the Pull Request

Add a new detection rule about detection of incoming and accepted connections via the remote connection tool AnyDesk. This could be a sign of persistence and C2 activities.

Changelog

Example Log Event

I am not sure to have used the correct field direction. Im using Elastic and for sysmon events the associated field is network.direction.

This field is fundamental for the rule.

Fixed Issues

SigmaHQ Rule Creation Conventions

  • If your PR adds new rules, please consider following and applying these conventions

@github-actions github-actions bot added Rules Windows Pull request add/update windows related rules labels Jul 2, 2024
@dan21san
Copy link
Contributor Author

dan21san commented Jul 2, 2024

I think the problem is direction: 'ingress' . How this could be fix?

@nasbench
Copy link
Member

nasbench commented Jul 2, 2024

I think the problem is direction: 'ingress' . How this could be fix?

The rule is using fields and values generated by the elastic agent / siem. This would qualify this as a rule that can "only" be used by such a tool. Can you link to another EDR or collection agent that have these kind of enrichment?

It doesn't make sense (most of the time) to create a Sigma rule that can only be converted to a single backend. (we try to avoid those when possible).

I'll evaluate and review the usefulness of the rule regardless and see if we can add it to the set with a definition section. But please try and link other solutions :)

@nasbench nasbench added Work In Progress Some changes are needed Author Input Required changes the require information from original author of the rules labels Jul 2, 2024
@nasbench nasbench self-assigned this Jul 2, 2024
@nasbench nasbench marked this pull request as draft July 3, 2024 12:44
@nasbench nasbench added the Not-Possible The rule cannot be accepted or implemented from the information provided label Aug 1, 2024
@nasbench
Copy link
Member

nasbench commented Aug 1, 2024

Closing this due to inactivity for now.

Further information is required and internal discussion needs to be made

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

Labels

Author Input Required changes the require information from original author of the rules Not-Possible The rule cannot be accepted or implemented from the information provided Rules Windows Pull request add/update windows related rules Work In Progress Some changes are needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants