Skip to content

dnsdist: Include the address of the backend in 'relayed to' messages #11578

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

Merged
merged 1 commit into from
Apr 29, 2022

Conversation

rgacogne
Copy link
Member

Short description

Not all backends have a name.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • 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
Copy link
Member Author

CI failure in test-auth-backend (gsqlite3, coscale/docker-sleep) is unrelated:

  + cat supermaster-signed/diff
  --- /home/runner/work/pdns/pdns/regression-tests.nobackend/supermaster-signed/expected_result	2022-04-26 08:10:09.436636317 +0000
  +++ /home/runner/work/pdns/pdns/regression-tests.nobackend/supermaster-signed/real_result	2022-04-26 08:16:18.881140149 +0000
  @@ -3,6 +3,3 @@
   Set 'example.com' meta TSIG-ALLOW-AXFR = tsig.com
   Set 'test.com' meta TSIG-ALLOW-AXFR = tsig.com
   Set 'example.com' meta AXFR-MASTER-TSIG = tsig.com
  -Added to queue
  -Received notification response with error from 127.0.0.2:53: Query Refused
  -For: 'test.com'
  + exit 1

@rgacogne rgacogne merged commit 24f8394 into PowerDNS:master Apr 29, 2022
@rgacogne rgacogne deleted the ddist-relayed-msg branch April 29, 2022 06:13
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.

1 participant