Skip to content

Unbound: convert advanced page to MVC#6050

Merged
swhite2 merged 3 commits intoopnsense:masterfrom
swhite2:unbound_advanced
Sep 27, 2022
Merged

Unbound: convert advanced page to MVC#6050
swhite2 merged 3 commits intoopnsense:masterfrom
swhite2:unbound_advanced

Conversation

@swhite2
Copy link
Member

@swhite2 swhite2 commented Sep 27, 2022

  • adds the Log Reply and Tag Queries and Replies options to enhance logging capabilities as well.

Copy link
Member

@fichtner fichtner left a comment

Choose a reason for hiding this comment

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

LGTM

@swhite2 swhite2 merged commit f3e0e06 into opnsense:master Sep 27, 2022
swhite2 added a commit that referenced this pull request Sep 30, 2022
Small addition to #6050:

- Move private/insecure domains to advanced as its intended use has a larger scope than DNSBLs only. Fixes #5256
- Extends the migration to also include these domains.
- leftover cleanup of legacy settings in migration.
- Adds the `serve-expired-reply-ttl`, `serve-expired-ttl`, `serve-expired-ttl-reset`, `serve-expired-client-timeout` options. These options are hidden until the `serve-expired` checkbox is checked, and indented to signify a relationship.
- Removes all dropdowns and instead provides numeric fields to input raw values for more control and less "guessing" of what is acceptable.
- Removes default settings to prevent mismatches with upstream in the future. It's probably best to refer to the Unbound documentation in our own documentation.
- Previously, `rrset-cache-size` and `outgoing-range` were implicitly set. These are now input fields. The migration code will take care of legacy setting assumptions.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants