/pdns

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

dnsdist: Optionally export CNAME over protobuf #4709

Closed
rgacogne opened this Issue Nov 24, 2016 · 0 comments

Comments

Assignees
No one assigned
Labels
dnsdist feature request
Projects
None yet
Milestone
No milestone
1 participant
@rgacogne
@rgacogne
Member

rgacogne commented Nov 24, 2016

  • Program: dnsdist
  • Issue type: Feature request

Short description

dnsdist tries very hard to parse as little as possible of DNS responses, and for this reason only exports A and AAAA records over protobuf, whereas the recursor also exports CNAME ones. It would be nice to be able to log these, at a slightly increased CPU cost.

We could for example add a parseCNAME option to newRemoteLogger():

newRemoteLogger(address:port [, timeout=2, maxQueuedEntries=100, reconnectWaitTime=1, parseCNAME=false])

@rgacogne rgacogne added dnsdist feature request labels Nov 24, 2016

@rgacogne rgacogne referenced this issue Dec 16, 2016

Merged

dnsdist: Add an option to export CNAME records over protobuf #4776

5 of 5 tasks complete

@rgacogne rgacogne closed this in #4776 Jan 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment