Skip to content

Conversation

TochusC
Copy link
Contributor

@TochusC TochusC commented Oct 17, 2024

This pull request introduces support for RRSIG and DNSKEY record types in the DNS layer. The changes include the addition of these record types to the DNSType enumeration, as well as the implementation of their respective structures, encoding methods, and size calculations. This enhancement allows for better handling of DNSSEC-related records, improving the overall functionality of the DNS module.

This was referenced Oct 17, 2024
@TochusC
Copy link
Contributor Author

TochusC commented Oct 17, 2024

I have tested it in my environment, and it appears to work fine.
db2c8181-c868-4ccb-8300-80ced266fb7b

@TochusC
Copy link
Contributor Author

TochusC commented Oct 17, 2024

DNSKEY works fine too.
image

@TochusC
Copy link
Contributor Author

TochusC commented Oct 17, 2024

Issue can be found here: #91

@TochusC TochusC closed this Oct 17, 2024
@TochusC TochusC reopened this Oct 17, 2024
Copy link
Contributor

@mosajjal mosajjal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good. can you please added a few packets with and without DNSSEC to the test files as well.

@TochusC
Copy link
Contributor Author

TochusC commented Oct 21, 2024

OK, I will finish it asap.

test: add RRSIG and DNSKEY test cases in test file.
@TochusC TochusC requested a review from mosajjal October 24, 2024 07:34
@TochusC
Copy link
Contributor Author

TochusC commented Oct 24, 2024

Done.
I am working to add more DNSSEC types like DS, NSEC, NSEC3.

@mosajjal mosajjal merged commit f9b8e3b into gopacket:master Oct 25, 2024
1 check passed
wader pushed a commit to wader/gopacket that referenced this pull request Oct 25, 2024
* feat: add RRSIG and DNSKEY record types to DNS layer

* feat: enhance DNS record types.

* feat: implement RRSIG and DNSKEY decoding and encoding
test: add RRSIG and DNSKEY test cases in test file.
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