Skip to content

Conversation

@rgacogne
Copy link
Member

@rgacogne rgacogne commented Aug 3, 2017

Short description

Document the new features introduced in #5396.

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 rgacogne added this to the dnsdist-1.2.0 milestone Aug 3, 2017

:returns: true if the DO bit was set, false otherwise

.. classmethod:: DNSQuestion.getTag(key) -> string
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be DNSQuestion:getTag(key) (also for the other newly added functions?)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed it should, I blindly copied the DNSQuestion.getDO() typo above. I'll fix it, thanks!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed!

@rgacogne rgacogne merged commit 53ce163 into PowerDNS:master Aug 14, 2017
@rgacogne rgacogne deleted the dist-doc-tags branch August 14, 2017 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants