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: Filter on opcode, records count/type, trailing data #3936

Merged
merged 2 commits into from Jun 7, 2016

Conversation

Projects
None yet
3 participants
@rgacogne
Member

rgacogne commented Jun 6, 2016

  • Add OpcodeRule() to filter on opcode + DNSOpcode.* Lua values
  • Add TrailingDataRule() to filter queries with trailing data
  • Add RecordsCountRule(section, minCount, maxCount) to match on
    the number of records in a given section
  • Add RecordsTypeCountRule(section, type, minCount, maxCount) to
    match on the number of records of type type in a given section
  • Add DNSSection.* Lua values
  • Add DNSClass.* Lua values
dnsdist: Filter on opcode, records count/type, trailing data
* Add `OpcodeRule()` to filter on opcode + DNSOpcode.* Lua values
* Add `TrailingDataRule()` to filter queries with trailing data
* Add `RecordsCountRule(section, minCount, maxCount)` to match on
the number of records in a given section
* Add `RecordsTypeCountRule(section, type, minCount, maxCount)` to
match on the number of records of type `type` in a given section
* Add DNSSection.* Lua values
* Add DNSClass.* Lua values
for(size_t n = 0; n < numrecords; ++n) {
dpm.skipLabel();
/* const uint16_t dnstype */ dpm.get16BitInt();

This comment has been minimized.

@zeha

zeha Jun 6, 2016

Collaborator

i'd suggest not having multi-line comment syntax and actual code on the same line

@ahupowerdns ahupowerdns merged commit cead95c into PowerDNS:master Jun 7, 2016

1 check passed

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

@rgacogne rgacogne deleted the rgacogne:dnsdist-more-filtering-options branch Jun 7, 2016

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