From e28564dbde4f922a6a8263e3dea32d56b60b5b5b Mon Sep 17 00:00:00 2001
From: tbrehm <t.brehm@ispconfig.org>
Date: Wed, 16 Jun 2010 11:41:46 -0400
Subject: [PATCH] Implemented: FS#468 - Client name conversion in FTP user too restricted

---
 server/plugins-available/bind_plugin.inc.php |   67 +++++++++++++++++++++++----------
 1 files changed, 46 insertions(+), 21 deletions(-)

diff --git a/server/plugins-available/bind_plugin.inc.php b/server/plugins-available/bind_plugin.inc.php
index 5aebc37..3c084b3 100644
--- a/server/plugins-available/bind_plugin.inc.php
+++ b/server/plugins-available/bind_plugin.inc.php
@@ -83,35 +83,42 @@
 	function soa_update($event_name,$data) {
 		global $app, $conf;
 		
+		//* Load libraries
+		$app->uses("getconf,tpl");
+		
 		//* load the server configuration options
-		$app->uses("getconf");
 		$dns_config = $app->getconf->get_server_config($conf["server_id"], 'dns');
 		
 		//* Write the domain file
-		$tpl = new tpl();
-		$tpl->newTemplate("bind_pri.domain.master");
+		if(!empty($data['new']['id'])) {
+			$tpl = new tpl();
+			$tpl->newTemplate("bind_pri.domain.master");
 		
-		$zone = $data['new'];
-		$tpl->setVar($zone);
+			$zone = $data['new'];
+			$tpl->setVar($zone);
 		
-		$records = $app->db->queryAllRecords("SELECT * FROM dns_rr WHERE zone = ".$zone['id']);
-		$tpl->setLoop('records',$records);
+			$records = $app->db->queryAllRecords("SELECT * FROM dns_rr WHERE zone = ".$zone['id']." AND active = 'Y'");
+			$tpl->setLoop('zones',$records);
+			
 		
-		$filename = escapeshellcmd($dns_config['bind_zonefiles_dir'].'/pri.'.$zone['origin']);
-		file_put_contents($filename,$tpl->grab());
-		exec('chown '.escapeshellcmd($dns_config['bind_user']).':'.escapeshellcmd($dns_config['bind_group']).' '.$filename);
-		unset($tpl);
-		unset($records);
-		unset($zone);
+			$filename = escapeshellcmd($dns_config['bind_zonefiles_dir'].'/pri.'.substr($zone['origin'],0,-1));
+			$app->log("Writing BIND domain file: ".$filename,LOGLEVEL_DEBUG);
+			file_put_contents($filename,$tpl->grab());
+			exec('chown '.escapeshellcmd($dns_config['bind_user']).':'.escapeshellcmd($dns_config['bind_group']).' '.$filename);
+			unset($tpl);
+			unset($records);
+			unset($records_out);
+			unset($zone);
+		}
 		
 		//* rebuild the named.conf file if the origin has changed or when the origin is inserted.
-		if($this->action == 'insert' || $data['old']['origin'] != $data['new']['origin']) {
-			$this->write_named_conf($data,$dns_config);
-		}
+		//if($this->action == 'insert' || $data['old']['origin'] != $data['new']['origin']) {
+		$this->write_named_conf($data,$dns_config);
+		//}
 		
 		//* Delete old domain file, if domain name has been changed
 		if($data['old']['origin'] != $data['new']['origin']) {
-			$filename = $dns_config['bind_zonefiles_dir'].'/pri.'.$data['old']['origin'];
+			$filename = $dns_config['bind_zonefiles_dir'].'/pri.'.substr($data['old']['origin'],0,-1);
 			if(is_file($filename)) unset($filename);
 		}
 		
@@ -124,15 +131,16 @@
 		global $app, $conf;
 		
 		//* load the server configuration options
-		$app->uses("getconf");
+		$app->uses("getconf,tpl");
 		$dns_config = $app->getconf->get_server_config($conf["server_id"], 'dns');
 		
 		//* rebuild the named.conf file
 		$this->write_named_conf($data,$dns_config);
 		
 		//* Delete the domain file
-		$filename = $dns_config['bind_zonefiles_dir'].'/pri.'.$data['old']['origin'];
-		if(is_file($filename)) unset($filename);
+		$zone_file_name = $dns_config['bind_zonefiles_dir'].'/pri.'.substr($data['old']['origin'],0,-1);
+		if(is_file($zone_file_name)) unlink($zone_file_name);
+		$app->log("Deleting BIND domain file: ".$zone_file_name,LOGLEVEL_DEBUG);
 		
 		//* Reload bind nameserver
 		$app->services->restartServiceDelayed('bind','reload');
@@ -180,14 +188,31 @@
 	function write_named_conf($data, $dns_config) {
 		global $app, $conf;
 		
-		$zones = $app->db->queryAllRecords("SELECT origin FROM dns_soa WHERE active = 'Y'");
+		$tmps = $app->db->queryAllRecords("SELECT origin, xfer, also_notify FROM dns_soa WHERE active = 'Y'");
+		$zones = array();
+
+		foreach($tmps as $tmp) {
+			
+			$options = '';
+			if(trim($tmp['xfer']) != '') $options .= '        allow-transfer {'.str_replace(',',';',$tmp['xfer']).";};\n";
+			if(trim($tmp['also_notify']) != '') $options .= '        also-notify {'.str_replace(',',';',$tmp['also_notify']).";};\n";
+			
+			$zones[] = array(	'zone' => substr($tmp['origin'],0,-1),
+								'zonefile_path' => $dns_config['bind_zonefiles_dir'].'/pri.'.substr($tmp['origin'],0,-1),
+								'options' => $options
+							);
+		}
 		
 		$tpl = new tpl();
 		$tpl->newTemplate("bind_named.conf.local.master");
 		$tpl->setLoop('zones',$zones);
 		
 		file_put_contents($dns_config['named_conf_local_path'],$tpl->grab());
+		$app->log("Writing BIND named.conf.local file: ".$dns_config['named_conf_local_path'],LOGLEVEL_DEBUG);
+		
 		unset($tpl);
+		unset($zones);
+		unset($tmps);
 		
 	}
 	

--
Gitblit v1.9.1