Skip to content

Commit ea97ae1

Browse files
author
A. Täffner
committed
Zone-Checking improved
Implements a colored background if bind fails to load the zone due to errors Implements a status text if errors occured Implements not removing a zone if errors occur. instead old zone stays loaded
1 parent 698ed15 commit ea97ae1

File tree

6 files changed

+52
-5
lines changed

6 files changed

+52
-5
lines changed

‎interface/web/dns/form/dns_soa.tform.php‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,20 @@
8080
'width' => '30',
8181
'maxlength' => '255'
8282
),
83+
'status' => array (
84+
'datatype' => 'VARCHAR',
85+
'formtype' => 'TEXT',
86+
'default' => 'PENDING',
87+
'value' => 'PENDING'
88+
),
89+
'status_txt' => array (
90+
'datatype' => 'VARCHAR',
91+
'formtype' => 'TEXTAREA',
92+
'default' => '',
93+
'value' => '',
94+
'width' => '30',
95+
'maxlength' => '10000'
96+
),
8397
'origin' => array (
8498
'datatype' => 'VARCHAR',
8599
'formtype' => 'TEXT',

‎interface/web/dns/list/dns_soa.list.php‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,27 @@
6161
'width' => "",
6262
'value' => array('Y' => "<div id=\"ir-Yes\" class=\"swap\"><span>".$app->lng('yes_txt')."</span></div>", 'N' => "<div class=\"swap\" id=\"ir-No\"><span>".$app->lng('no_txt')."</span></div>"));
6363

64+
65+
$liste["item"][] = array( 'field' => "status",
66+
'datatype' => "VARCHAR",
67+
'formtype' => "SELECT",
68+
'op' => "=",
69+
'prefix' => "",
70+
'suffix' => "",
71+
'width' => "",
72+
'value' => array('OK' => '', 'ERROR' => 'background-color:red', 'PENDING' => 'background-color:yellow'));
73+
74+
75+
$liste["item"][] = array( 'field' => "status_txt",
76+
'datatype' => "VARCHAR",
77+
'formtype' => "SELECT",
78+
'op' => "=",
79+
'prefix' => "",
80+
'suffix' => "",
81+
'width' => "",
82+
'value' => '');
6483

84+
6585
$liste["item"][] = array( 'field' => "server_id",
6686
'datatype' => "VARCHAR",
6787
'formtype' => "SELECT",

‎interface/web/dns/templates/dns_soa_admin_list.htm‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ <h1><tmpl_var name="list_head_txt"></h1>
5757
<tbody>
5858
<tmpl_loop name="records">
5959
<tr>
60-
<td><a href="#" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="active"}</td>
60+
<td style="{tmpl_var name="status"}"><a href="#" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="active"}</td>
6161
<td><a href="#" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="sys_groupid"}</a></td>
6262
<td><a href="#" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="server_id"}</a></td>
6363
<td><a href="#" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="origin"}</a></td>

‎interface/web/dns/templates/dns_soa_edit.htm‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,11 @@ <h1><tmpl_var name="list_head_txt"></h1>
55

66

77
<legend>DNS Zone</legend>
8+
<tmpl_if name="status_txt">
9+
<div class="alert alert-danger clear">
10+
<div style="white-space: pre;" class="alert-content"><tmpl_var name="status_txt"></div>
11+
</div>
12+
</tmpl_if>
813
<tmpl_if name="is_admin">
914
<div class="form-group">
1015
<tmpl_if name="edit_disabled">
@@ -127,6 +132,7 @@ <h1><tmpl_var name="list_head_txt"></h1>
127132

128133

129134
<input type="hidden" name="id" value="{tmpl_var name='id'}">
135+
<input type="hidden" name="status" value="PENDING">
130136
<input name="serial" type="hidden" value="{tmpl_var name='serial'}">
131137

132138
<div class="clear"><div class="right">

‎interface/web/dns/templates/dns_soa_list.htm‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ <h1><tmpl_var name="list_head_txt"></h1>
5555
<tbody>
5656
<tmpl_loop name="records">
5757
<tr>
58-
<td><a href="#" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="active"}</td>
58+
<td style="{tmpl_var name="status"}"><a href="#" title="{tmpl_var name="status_txt"}" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="active"}</td>
5959
<td><a href="#" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="server_id"}</a></td>
6060
<td><a href="#" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="origin"}</a></td>
6161
<td><a href="#" data-load-content="dns/dns_soa_edit.php?id={tmpl_var name='id'}">{tmpl_var name="ns"}</a></td>

‎server/plugins-available/bind_plugin.inc.php‎

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -123,22 +123,28 @@ function soa_update($event_name, $data) {
123123
$filename = escapeshellcmd($dns_config['bind_zonefiles_dir'].'/pri.'.str_replace("/", "_", substr($zone['origin'], 0, -1)));
124124
}
125125

126-
file_put_contents($filename, $tpl->grab());
126+
file_put_contents($filename.'.pending', $tpl->grab());
127127
chown($filename, escapeshellcmd($dns_config['bind_user']));
128128
chgrp($filename, escapeshellcmd($dns_config['bind_group']));
129129

130130
//* Check the zonefile
131131
if(is_file($filename.'.err')) unlink($filename.'.err');
132-
exec('named-checkzone '.escapeshellarg($zone['origin']).' '.escapeshellarg($filename), $out, $return_status);
132+
$out=array();
133+
exec('/usr/sbin/named-checkzone '.escapeshellarg($zone['origin']).' '.escapeshellarg($filename.'.pending').' 2>&1', $out, $return_status);
134+
$statustext='';
135+
foreach ($out as $line) $statustext .= $line."\n";
133136
if($return_status === 0) {
134137
$app->log("Writing BIND domain file: ".$filename, LOGLEVEL_DEBUG);
138+
$app->db->query('UPDATE dns_soa SET status=\'OK\', status_txt=\'\' WHERE id='.$data['new']['id']);
139+
rename($filename.'.pending', $filename);
135140
} else {
136141
if($dns_config['disable_bind_log'] === 'y') {
137142
$app->log("Writing BIND domain file failed: ".$filename." ".implode(' ', $out), LOGLEVEL_DEBUG);
138143
} else {
139144
$app->log("Writing BIND domain file failed: ".$filename." ".implode(' ', $out), LOGLEVEL_WARN);
140145
}
141-
rename($filename, $filename.'.err');
146+
$app->db->query('UPDATE dns_soa SET status=\'ERROR\', status_txt=\''.str_replace(array('"', '\''), '', $statustext).'\' WHERE id='.$data['new']['id']);
147+
rename($filename.'.pending', $filename.'.err');
142148
}
143149
unset($tpl);
144150
unset($records);
@@ -163,6 +169,7 @@ function soa_update($event_name, $data) {
163169

164170
if(is_file($filename)) unlink($filename);
165171
if(is_file($filename.'.err')) unlink($filename.'.err');
172+
if(is_file($filename.'.pending')) unlink($filename.'.pending');
166173
}
167174

168175
//* Restart bind nameserver if update_acl is not empty, otherwise reload it

0 commit comments

Comments
 (0)