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: Fix dynblocks over TCP, allow refusing dyn blocked queries #4518

Merged
merged 4 commits into from Nov 2, 2016

Conversation

Projects
None yet
1 participant
@rgacogne
Member

rgacogne commented Sep 30, 2016

This PR adds a new feature, allowing to 'refuse' dynamically blocked queries instead of dropping them, and fixes two issues:

  • we stored the query header instead of the response header in the response ring for TCP queries
  • we mixed "real" and monotonic time for TCP queries, storing non-sense in the response ring

rgacogne added some commits Sep 30, 2016

@rgacogne rgacogne added this to the dnsdist-1.1.0 milestone Sep 30, 2016

@rgacogne rgacogne merged commit 6bb2663 into PowerDNS:master Nov 2, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@rgacogne rgacogne deleted the rgacogne:dnsdist-dynblock-action branch Nov 2, 2016

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