Thanks to visit codestin.com
Credit goes to github.com

Skip to content

auth: better advice when creating zones#16670

Merged
miodvallat merged 3 commits intoPowerDNS:masterfrom
miodvallat:frontline
Feb 9, 2026
Merged

auth: better advice when creating zones#16670
miodvallat merged 3 commits intoPowerDNS:masterfrom
miodvallat:frontline

Conversation

@miodvallat
Copy link
Contributor

@miodvallat miodvallat commented Dec 19, 2025 •
edited
Loading

Short description

This adds the ability for pdns_control to update the zone cache, either refreshing the whole cache or a given zone.

Turns out this was already possible using pdns_control rediscover. So drop that change, and replace it with pdnsutil giving you that advice when creating a zone, if the configuration has the zone cache enabled.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • read and accepted the Developer Certificate of Origin document, including the AI Policy, and added a "Signed-off-by" to my commits
  • 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)

@coveralls
Copy link

coveralls commented Dec 19, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build 21666859671

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 7 of 12 (58.33%) changed or added relevant lines in 1 file are covered.
  • 30 unchanged lines in 9 files lost coverage.
  • Overall coverage increased (+0.001%) to 71.547%
Changes Missing Coverage Covered Lines Changed/Added Lines %
pdns/dynhandler.cc 7 12 58.33%
Files with Coverage Reduction New Missed Lines %
pdns/backends/gsql/gsqlbackend.hh 1 97.77%
pdns/misc.cc 2 61.62%
pdns/recursordist/rec-main.cc 2 64.88%
pdns/axfr-retriever.cc 3 67.07%
pdns/recursordist/syncres.cc 3 81.0%
pdns/remote_logger.cc 3 58.12%
pdns/recursordist/pdns_recursor.cc 4 75.79%
pdns/recursordist/lwres.cc 6 66.0%
pdns/recursordist/test-syncres_cc1.cc 6 80.4%
Totals Coverage Status
Change from base Build 21664614329: 0.001%
Covered Lines: 129248
Relevant Lines: 165673
💛 - Coveralls

@miodvallat miodvallat changed the title auth: let pdns_control purge also affect zone cache auth: better advice when creating zones Dec 22, 2025
@miodvallat miodvallat force-pushed the frontline branch 2 times, most recently from f766a38 to 231b4c8 Compare December 22, 2025 14:02
rgacogne
rgacogne reviewed Dec 23, 2025
View reviewed changes
rgacogne
rgacogne previously approved these changes Dec 29, 2025
View reviewed changes
@miodvallat
Do not print recommendations if quiet.
acc1343 
There are people out there using shell scripts to make nails out of
pdnsutil, do not make pdnsutil output parsing more difficult.

Signed-off-by: Miod Vallat <[email protected]>
@miodvallat miodvallat merged commit f1614a5 into PowerDNS:master Feb 9, 2026
165 of 168 checks passed
@miodvallat miodvallat deleted the frontline branch February 9, 2026 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rgacogne rgacogne rgacogne approved these changes

Assignees

No one assigned

Labels

auth

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@miodvallat @coveralls @rgacogne