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

tinydns: report broken content that causes errors #5791

Merged
merged 1 commit into from Oct 9, 2017

Conversation

Projects
None yet
3 participants
@Habbie
Member

Habbie commented Oct 8, 2017

Short description

When throwing because of bogus content in the tinydns database, report the offending name+type so the admin can find the offending record.

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)
  • checked that this code was merged to master
@Habbie

This comment has been minimized.

Show comment
Hide comment
@Habbie

Habbie Oct 8, 2017

Member

@thomas-mangin this is for you!

Member

Habbie commented Oct 8, 2017

@thomas-mangin this is for you!

@Habbie Habbie added this to the auth-4.1.0 milestone Oct 8, 2017

@thomas-mangin

This comment has been minimized.

Show comment
Hide comment
@thomas-mangin

thomas-mangin Oct 8, 2017

Thank you :-)

thomas-mangin commented Oct 8, 2017

Thank you :-)

@zeha

zeha approved these changes Oct 8, 2017

@Habbie Habbie merged commit e4986cf into PowerDNS:master Oct 9, 2017

1 check passed

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

@Habbie Habbie deleted the Habbie:tinydns-bogus-reporting branch Oct 9, 2017

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