[Bug 288689] dns/dnsmasq: test config before restart

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=288689

            Bug ID: 288689
           Summary: dns/dnsmasq: test config before restart
           Product: Ports & Packages
           Version: Latest
          Hardware: Any
                OS: Any
            Status: New
          Severity: Affects Only Me
          Priority: ---
         Component: Individual Port(s)
          Assignee: mandree@FreeBSD.org
          Reporter: gerrit.kuehn@aei.mpg.de
             Flags: maintainer-feedback?(mandree@FreeBSD.org)
          Assignee: mandree@FreeBSD.org

In everyday operation, it sometimes happens that the dnsmasq.conf file is
changed in a way the prevents dnsmasq from parsing ist properly. In such a
case, issuing "service dnsmasq restart" gives back an error message (that is
sometimes hard to spot because of many other output) and leaves the user with a
dns/dhcp service that is not running and not startable manually (the config
file needs to be fixed first).

I cannot imagine any situation where this would be the desired result,
especially with a crucial network service as dns or dhcp.
Thus, I suggest to add and additional check of the config file before
restarting the service. This is easily done with dnsmasq itself as it comes
with a test function built-in:

> /usr/local/sbin/dnsmasq -C /usr/local/etc/dnsmasq.conf --test

If adding this test as mandatory for a restart is considered a violation of
POLA, I would suggest to add an extra rc option that allows enabling the test.
In either case, I would be willing to help working on this, if desired.

-- 
You are receiving this mail because:
You are the assignee for the bug.