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

fix: Removed deprecated named options #4685

Merged
merged 1 commit into from
May 26, 2023

Conversation

nicolatimeus
Copy link
Contributor

Note: We are using the Conventional Commits convention for our pull request titles. Please take a look at the PR title format document for the supported types and scopes.

Brief description of the PR. [e.g. Added null check on object to avoid NullPointerException]

  • Removes the dnssec-enable and dnssec-lookaside options which have been deprecated in bind 9.16 [1] and since 9.18 setting them is considered a configuration failure [2] that will prevent named from starting.

  • Sets dnssec-validation to no, since it is equivalent to setting it to yes without including a trust-anchors statement (no dnssec validation is performed in both cases).

[1] https://bind9.readthedocs.io/en/v9.16.16/notes.html#notes-for-bind-9-16-0
[2] https://bind9.readthedocs.io/en/v9_18_6/notes.html?highlight=dnssec-enable#notes-for-bind-9-18-0
[3] https://bind9.readthedocs.io/en/v9.18.13/dnssec-guide.html#dnssec-validation-explained

@MMaiero MMaiero merged commit 28941cb into eclipse-kura:develop May 26, 2023
github-actions bot pushed a commit that referenced this pull request May 26, 2023
Signed-off-by: Nicola Timeus <[email protected]>
(cherry picked from commit 28941cb)
MMaiero pushed a commit that referenced this pull request May 26, 2023
fix: Removed deprecated named options (#4685)

Signed-off-by: Nicola Timeus <[email protected]>
(cherry picked from commit 28941cb)

Co-authored-by: nicolatimeus <[email protected]>
@nicolatimeus nicolatimeus deleted the fix_named-dnssec branch March 22, 2024 08:05
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