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

Proper parent identification for dynamically entered local zones #1076

Merged
merged 2 commits into from
May 24, 2024

Conversation

gthess
Copy link
Member

@gthess gthess commented May 24, 2024

Addition of local zones dynamically via unbound-control was not finding the zone's parent correctly.
Fixes #1059.

  always_nxdomain. Addition of local_zones dynamically via
  unbound-control was not finding the zone's parent correctly.
@gthess gthess requested a review from wcawijngaards May 24, 2024 10:07
@gthess gthess self-assigned this May 24, 2024
Copy link
Member

@wcawijngaards wcawijngaards left a comment

Choose a reason for hiding this comment

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

The code is nice. Perhaps rename initparents to some globally unique name, now that it is exported out of the localzone code file.

@gthess
Copy link
Member Author

gthess commented May 24, 2024

Thanks @wcawijngaards , I agree about the renaming.

@gthess gthess merged commit 6f030e9 into master May 24, 2024
1 check passed
@gthess gthess deleted the bugfix/proper-parent-dynamic-localzone branch May 24, 2024 13:21
jedisct1 added a commit to jedisct1/unbound that referenced this pull request Jun 16, 2024
* nlnet/master: (35 commits)
  - Add unit test for validation of repeated use of a DNAME record.
  - Fix validation for repeated use of a DNAME record.
  - Fix typos for 'the the' in text.
  - Fix memory leak in setup of dsa sig.
  - Skip unbound-dnstap-socket unit test when not compiled with   --enable-debug.
  - Fix to squelch connection reset by peer errors from log. And fix   that the tcp read errors are labeled as initial for the first calls.
  - Fix memory leak on exit for unbound-dnstap-socket; creates false negatives   during testing.
  - Fix memory leak when reload_keep_cache is used and num-threads changes.
  - Enable AddressSanitizer error detection in tdir tests.
  - Fix for NLnetLabs#1079: fix RPZ taglist in iterator callback that no client   info is like no taglist intersection.
  - Fix NLnetLabs#1079: tags from tagged rpz zones are no longer honored after   upgrade from 1.19.3 to 1.20.0.
  Changelog note for NLnetLabs#1078. - Merge NLnetLabs#1078: Only check old pid if no username.
  Only check old pid if no username
  - Update patch to remove 'command' shell builtin and update error   text.
  unbound-control-setup: check openssl
  - Fix unused variable warning on compilation with no thread support.
  - Fix spelling of tcp-idle-timeout docs, from Michael Tokarev.
  - Fix to enable that SERVFAIL is cached, for a short period, for more   cases. In the cases where limits are exceeded.
  Changelog entry for NLnetLabs#1059: - Fix NLnetLabs#1059: Intermittent DNS blocking failure with local-zone and   always_nxdomain. Addition of local_zones dynamically via   unbound-control was not finding the zone's parent correctly.
  Proper parent identification for dynamically entered local zones (NLnetLabs#1076)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Intermittent DNS blocking failure with local-zone and always_nxdomain
2 participants