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

Add XPF support #6220

merged 11 commits into from Jan 23, 2018

Add XPF support #6220

merged 11 commits into from Jan 23, 2018


Copy link

@pieterlexis pieterlexis commented Jan 23, 2018

Short description

This is a rebased and conflicts-fixed version of #5594 with some additional changes:

  • Documentation fixes
  • Renaming of variables from Option Code to RR Code

Closes #5079
Closes #5594
Closes #5654


I have:

  • read the 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)
Habbie and others added 11 commits Jul 25, 2017
It's a bit trickier for sdig, though.
make XPF code point configurable in sdig
remove XPF from dnsrecords and qtype
check argument count for sdig modifiers
And add some docs.
@pieterlexis pieterlexis force-pushed the pieterlexis:ddist-xpf branch from 8571376 to c85f69a Jan 23, 2018
@pieterlexis pieterlexis merged commit 2f4ccf9 into PowerDNS:master Jan 23, 2018
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
@pieterlexis pieterlexis deleted the pieterlexis:ddist-xpf branch Jan 23, 2018
@pieterlexis pieterlexis added this to the dnsdist-1.3.0 milestone Mar 26, 2018
mnordhoff added a commit to mnordhoff/pdns that referenced this pull request Jul 15, 2019
PowerDNS#6220's title was "Add XPF support", but the only actual changes to Auth were to sdig and the test suite, and the changelog entry was misleading.

Reported by @spheron1.
@mnordhoff mnordhoff mentioned this pull request Jul 15, 2019
1 of 7 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
4 participants