unbound: Add support for DoH and DoT #5468
Conversation
|
Corresponding acme-client automation PR: opnsense/plugins#2747 |
| $dohdot_settings = ''; | ||
| if (isset($config['unbound']['enable_doh']) || isset($config['unbound']['enable_dot'])) { | ||
| $cert =& lookup_cert($config['unbound']['dohdot_cert']); | ||
| $chain = []; |
There was a problem hiding this comment.
Indentation on this and some following lines should be improved IMHO. From indentation I think the lines 356 & 357 are inside the if block. From the parenthesis these lines are not part of the ifblock.
|
Any chance of getting this into the 22.07 release? |
|
We're still working on some other features as well with Unbound, not sure if we can add DoH and DoT in the same time scheme. It's still on our radar, just careful about adding potential future issues in legacy code while migrating to our new model at the same time. |
|
@schreibubi @AdSchellevis Can i may ask the status of this PR? Get it upstreamed would be nice. Thanks for your contribution in any case! |
|
@jumbi77 situation hasn't changed, likely won't be merged in its current state, probably the functionality will be available at some point in time. |
Added additional values to the unbound statistics page which allow to monitor DoH and DoT queries.
|
Is the OPNsense Patch from 10th July 2022 compatible with the newest version, 23.1? |
|
opnsense-patch 9e310c1 still works for 23.1 |
|
I'm closing this since unmergable after @swhite2 moved general unbound settings page to MVC. |
|
Updated PR against latest master #6558 |
|
Any chance this could be reopened? The OP has kept their repo up to date. This would be useful. |
Add support for unbound to act as an DoH and DoT server
Added additional values to the unbound statistics page which allow to
monitor DoH and DoT query count.
Fixes #5104
Additional commit will be necessary to add the automation to restart unbound when the Acme certificate changes. Will submit a separate PR for that.