-
Notifications
You must be signed in to change notification settings - Fork 185
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
dns: Add support for IPv6 and variable length IPv4 headers. #2574
Conversation
Fixes: d772f2e ("dns: Parse DNS packet in user space") Signed-off-by: Mauricio Vásquez <mauriciov@microsoft.com>
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! I was wondering if we could add some tests for this in the CI.
proto = load_byte(skb, | ||
ETH_HLEN + offsetof(struct iphdr, protocol)); |
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.
nit: This could be taken from iph->protocol
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.
I'd prefer to handle this on a similar PR to #2579.
Support IPv6 nameservers and extend parsing logic to support IPv4 options. Signed-off-by: Mauricio Vásquez <mauriciov@microsoft.com>
f43786f
to
67bfa60
Compare
I fear it's too complicated. We'll need to have a cluster with ipv6 enabled, and/or being able to run containers with ipv6. I can open an issue to handle this later on. |
Support IPv6 nameservers and extend parsing logic to support IPv4 options.