Skip to content
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

Cherry-pick #14004 to 7.4: [SIEM] Add DNS enrichment to system/socket dataset #14065

Closed
wants to merge 1 commit into from

Conversation

adriansr
Copy link
Contributor

Cherry-pick of PR #14004 to 7.4 branch. Original message:

This patch adds DNS monitoring to the system/socket dataset so that events can be enriched with domain names.

Its been implemented using af_packet so that filtering can be used at the kernel level and only DNS responses are copied to userspace.

In the future it might be possible to use a kprobes-based implementation which will make correlating DNS requests to processes much easier, but we need to find out how to filter DNS requests at kprobe level, given that sometimes is necessary to inspect the packet to obtain the source port.

@adriansr adriansr requested a review from a team as a code owner October 15, 2019 13:43
This patch adds DNS monitoring to the system/socket dataset so that
events can be enriched with domain names.

Its been implemented using af_packet so that filtering can be used at
the kernel level and only DNS responses are copied to userspace.

In the future it might be possible to use a kprobes-based implementation
which will make correlating DNS requests to processes much easier, but
we need to find out how to filter DNS requests at kprobe level, given
that sometimes is necessary to inspect the packet to obtain the source
port.

(cherry picked from commit ed9dba5)
@adriansr
Copy link
Contributor Author

Not backporting a feature to 7.4.1 :)

@adriansr adriansr closed this Oct 15, 2019
@adriansr adriansr deleted the backport_14004_7.4 branch October 15, 2019 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant