Add DNS Resolver to the list of services to be sync'd on HA, make sure it and DNS Forwarder are not enabled simultaneously. It fixes #4067

2 files changed, 14 insertions, 2 deletions

diff --git a/usr/local/www/system_hasync.php b/usr/local/www/system_hasync.php
index caeb79d..f77f666 100755
--- a/usr/local/www/system_hasync.php
+++ b/usr/local/www/system_hasync.php
@@ -322,11 +322,11 @@ include("head.inc");
 		</td>
 	</tr>
 	<tr valign="top">
-		<td width="22%" class="vncell">Synchronize DNS Forwarder</td>
+		<td width="22%" class="vncell">Synchronize DNS Forwarder / Resolver</td>
 		<td class="vtable">
 			<input id='synchronizednsforwarder' type='checkbox' name='synchronizednsforwarder' value='on' <?php if ($pconfig['synchronizednsforwarder'] === "on") echo "checked='checked'"; ?> />
 			<br />
-			When this option is enabled, this system will automatically sync the DNS Forwarder configuration to the other HA host when changes are made.
+			When this option is enabled, this system will automatically sync the DNS Forwarder and DNS Resolver configuration to the other HA host when changes are made.
 		</td>
 	</tr>
 	<tr valign="top">
diff --git a/usr/local/www/xmlrpc.php b/usr/local/www/xmlrpc.php
index a44789d..88dc537 100755
--- a/usr/local/www/xmlrpc.php
+++ b/usr/local/www/xmlrpc.php
@@ -211,6 +211,18 @@ function restore_config_section_xmlrpc($raw_params) {
 		}
 	}
 
+	/*
+	 * Make sure it doesn't end up with both dnsmasq and unbound enabled
+	 * simultaneously in secondary
+	 * */
+	if (isset($params[0]['unbound']['enable']) && isset($config['dnsmasq']['enable'])) {
+		unset($config['dnsmasq']['enable']);
+		services_dnsmasq_configure();
+	} else if (isset($params[0]['dnsmasq']['enable']) && isset($config['unbound']['enable'])) {
+		unset($config['unbound']['enable']);
+		services_unbound_configure();
+	}
+
         // For vip section, first keep items sent from the master
 	$config = array_merge_recursive_unique($config, $params[0]);