Project

General

Profile

Feature #4411

add validation of Unbound advanced configuration

Added by Chris Buechler over 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
DNS Resolver
Target version:
Start date:
02/11/2015
Due date:
% Done:

100%

Estimated time:

Description

Something similar to dnsmasq's advanced config validation would be helpful for Unbound, to prevent users from creating configs with invalid syntax.

Associated revisions

Revision 932711c7 (diff)
Added by Matthew Smith almost 4 years ago

Rearrange calls in unbound.inc so config text can be generated without writing the actual file and a syntax check can be run. Rearrange services_unbound.php to wait until input checking is complete before saving things to global config and run syntax check as part of input checking. Implements #4411.

History

#1 Updated by Paul K over 4 years ago

One thing you might consider is adding 'server:' as the last keyword in all configuration files that are included in the main config file. That would effectively "break out" from whatever directive was last used in the included file. This way you wouldn't have to add 'server: in the advanced options and might prevent other issues down the road.

#2 Updated by Chris Buechler about 4 years ago

  • Target version set to 2.3

#3 Updated by Jim Thompson almost 4 years ago

  • Assignee set to Matthew Smith

#4 Updated by Matthew Smith almost 4 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

#6 Updated by Matthew Smith almost 4 years ago

Thanks for your corrections. It looks like they were merged this morning.

#7 Updated by Chris Buechler almost 4 years ago

  • Status changed from Feedback to Resolved

works, thanks!

Also available in: Atom PDF