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

enhancement(dnstap source): add EDE code 30 to known EDE codes #21743

Merged
merged 2 commits into from
Nov 8, 2024

Conversation

esensar
Copy link
Contributor

@esensar esensar commented Nov 8, 2024

This adds EDE code 30 (Invalid Query Type), added in https://datatracker.ietf.org/doc/draft-ietf-dnsop-compact-denial-of-existence/04/ (which can also be seen in the EDE list -
https://www.iana.org/assignments/dns-parameters/dns-parameters.xhtml#extended-dns-error-codes) to the list of known ede codes, which generates the purpose string.

This adds EDE code 30 (Invalid Query Type), added in
https://datatracker.ietf.org/doc/draft-ietf-dnsop-compact-denial-of-existence/04/
(which can also be seen in the EDE list -
https://www.iana.org/assignments/dns-parameters/dns-parameters.xhtml#extended-dns-error-codes)
to the list of known ede codes, which generates the purpose string.
@esensar esensar requested a review from a team as a code owner November 8, 2024 16:24
@jszwedko jszwedko changed the title enhancement(dnsmsg-parser): add EDE code 30 to known EDE codes enhancement(vrl stdlib): add EDE code 30 to known EDE codes Nov 8, 2024
@jszwedko jszwedko changed the title enhancement(vrl stdlib): add EDE code 30 to known EDE codes enhancement(dnstap source): add EDE code 30 to known EDE codes Nov 8, 2024
@jszwedko jszwedko added this pull request to the merge queue Nov 8, 2024
Merged via the queue into vectordotdev:master with commit 246576b Nov 8, 2024
53 of 54 checks passed
@esensar esensar deleted the feat/ede-code-30 branch November 8, 2024 19:23
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