Skip to content

Draft: parse correlation rules - #33

Open
AnthonyAspen wants to merge 2 commits into
bradleyjkemp:mainfrom
AnthonyAspen:correlation-parser
Open

Draft: parse correlation rules#33
AnthonyAspen wants to merge 2 commits into
bradleyjkemp:mainfrom
AnthonyAspen:correlation-parser

Conversation

@AnthonyAspen

@AnthonyAspen AnthonyAspen commented Apr 21, 2023

Copy link
Copy Markdown

What's your thoughts about the correlation rules? I'd like to understand the state of the correlation rules branch. Have you had any plans on finishing it?

@AnthonyAspen AnthonyAspen changed the title WIP: parse correlation rules Draft: parse correlation rules Apr 21, 2023
@bradleyjkemp

Copy link
Copy Markdown
Owner

Hey @AnthonyAspen, I think in its current state it could probably be merged with only some small tidy up. That would only be for parsing correlation rules though, actually evaluating them is significantly more work.

Is just parsing the correlation rule format useful for you?

@AnthonyAspen

AnthonyAspen commented Apr 26, 2023

Copy link
Copy Markdown
Author

@bradleyjkemp

actually evaluating them is significantly more work.

Yeah It seems that we may need to refactor the evaluator to a "Matcher" interface, rather than using rules. Additionally, the correlation rules should have a storage interface to allow for state increments and sharing of state, in case one correlation rule relies on another.

Is just parsing the correlation rule format useful for you?

I think it'd be a great start. If you have something in your mind about implementing the correlation rules you can try to summarize it for me and I'll help


In brief, I came up with the idea of parsing the correlation rules after discovering that they have deprecated aggregations and appear to be planning to release the correlation rules in the future. However, the correlation rules are currently only in "version 2.0" of the specification, which makes me concerned that they could make significant changes that would render our (potential) work useless.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants