-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add registered_domain processor #13326
Add registered_domain processor #13326
Conversation
Pinging @elastic/siem |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor comment.
I don't like the name, as "registered" has implications, but can't come up with a better one either. It's funny how it seems there isn't a word for that. "etld_plus_one" won't make a catchy name for a processor.
@adriansr Yeah, the discussion around the name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM on the functionality.
As usual, don't rely on me for the Golang review.
Left 2 comments for small things
The `registered_domain` processor reads a field containing a hostname and then writes the "registered domain" contained in the hostname to the target field. For example, given `www.google.co.uk` the processor would output `google.co.uk`. In other words the "registered domain" is the effective top-level domain (`co.uk`) plus one level (`google`). This can be used to populate the ECS `dns.question.registered_domain` field.
fa85160
to
dcc2204
Compare
@adriansr I agree with you that the name isn't great. I chose it to match the ECS field name which went through much debate. |
The
registered_domain
processor reads a field containing a hostname and thenwrites the "registered domain" contained in the hostname to the target field.
For example, given
www.google.co.uk
the processor would outputgoogle.co.uk
.In other words the "registered domain" is the effective top-level domain
(
co.uk
) plus one level (google
).This can be used to populate the ECS
dns.question.registered_domain
field.