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: Implement tag-based routing, add missing LuaAction and LuaResponseAction #6037

Merged
merged 3 commits into from Dec 15, 2017

Conversation

Projects
None yet
1 participant
@rgacogne
Member

rgacogne commented Dec 1, 2017

Short description

It is now possible to assign a tag and a value to a query with TagAction() and to match on this tag and value with TagRule(). Tags are passed along to response so it's possible to match a response based on a tag set while processing the query. TagResponseAction() can also be used to set a tag, which can then be used with TagRule().

This PR also adds the missing LuaAction() and LuaResponseAction() that are especially needed since we deprecated addLuaAction() and addLuaResponseAction() in 1.2.0. It also fixes an issue that prevented access to the dnsheader from Lua in responses, for a reason I don't quite understand at the moment.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled and tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

rgacogne added some commits Dec 1, 2017

@rgacogne rgacogne added this to the dnsdist-1.3.0 milestone Dec 1, 2017

@rgacogne rgacogne merged commit 1bcd79d into PowerDNS:master Dec 15, 2017

1 check passed

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

@rgacogne rgacogne deleted the rgacogne:dnsdist-tags branch Dec 15, 2017

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