spamd, allow configuring interface to listen/intercept on

6 files changed, 60 insertions, 31 deletions

diff --git a/config/spamd/spamd.inc b/config/spamd/spamd.inc
index 71f20106..fd09837c 100644
--- a/config/spamd/spamd.inc
+++ b/config/spamd/spamd.inc
@@ -199,21 +199,35 @@ function spamd_generate_rules($type) {
 	switch($type) {
 	case 'nat':
 		$natrules .= "\n# spam table \n";
-                $wanif = get_real_interface("wan");
-                $natrules .= "table <whitelist> persist\n";
-                $natrules .= "table <blacklist> persist\n";
-                $natrules .= "table <spamd> persist\n";
-                if(file_exists("/var/db/whitelist.txt"))
-                        $natrules .= "table <spamd-white> persist file \"/var/db/whitelist.txt\"\n";
-                $natrules .= "rdr pass on {$wanif} proto tcp from <blacklist> to port smtp -> 127.0.0.1 port spamd\n";
-                $natrules .= "rdr pass on {$wanif} proto tcp from <spamd> to port smtp -> 127.0.0.1 port spamd\n";
-                $natrules .= "rdr pass on {$wanif} proto tcp from !<spamd-white> to port smtp -> 127.0.0.1 port spamd\n";
-                if($config['installedpackages']['spamdsettings']['config'])
-                        foreach($config['installedpackages']['spamdsettings']['config'] as $ss)
-                                $nextmta = $ss['nextmta'];
-                if($nextmta <> "") {
-                        $natrules .= "rdr pass on {$wanif} proto tcp from <spamd-white> to port smtp -> {$nextmta} port smtp\n";
-                }
+		$natrules .= "table <whitelist> persist\n";
+		$natrules .= "table <blacklist> persist\n";
+		$natrules .= "table <spamd> persist\n";
+		if(file_exists("/var/db/whitelist.txt"))
+				$natrules .= "table <spamd-white> persist file \"/var/db/whitelist.txt\"\n";
+
+		if (is_array($config['installedpackages']['spamdsettings']['config']))		
+			$spamdconfig = $config['installedpackages']['spamdsettings']['config']['0'];
+		else
+			$spamdconfig = $config['installedpackages']['spamdsettings']['config'];
+
+		if($spamdconfig) {
+			$nextmta = $spamdconfig['nextmta'];
+			$spamdbinds = explode(',', $spamdconfig['spamdbinds_array']);
+			if (is_array($spamdbinds)) {
+				foreach($spamdbinds as $interface) {
+					$wanif = get_real_interface($interface);
+					if (!isset($wanif))
+						continue;
+					$natrules .= "rdr pass on {$wanif} proto tcp from <blacklist> to port smtp -> 127.0.0.1 port spamd\n";
+					$natrules .= "rdr pass on {$wanif} proto tcp from <spamd> to port smtp -> 127.0.0.1 port spamd\n";
+					$natrules .= "rdr pass on {$wanif} proto tcp from !<spamd-white> to port smtp -> 127.0.0.1 port spamd\n";
+					if($nextmta <> "") {
+						$natrules .= "rdr pass on {$wanif} proto tcp from <spamd-white> to port smtp -> {$nextmta} port smtp\n";
+					}
+				}
+			}
+		}
+		$natrules .= "\n";
 
 		break;
 	}
diff --git a/config/spamd/spamd_db.php b/config/spamd/spamd_db.php
index e1d004f7..dab599b8 100644
--- a/config/spamd/spamd_db.php
+++ b/config/spamd/spamd_db.php
@@ -324,6 +324,11 @@ if (typeof getURL == 'undefined') {
 					<td class="list"></td>
 				</tr>
 <?php
+
+function formatspamddatetime($dt) {
+	return date("Y-m-d", $dt) . "<br/>" . date("H:i:s", $dt);
+}
+
 	if($filter) {
 		if($not) {
 			$fd = fopen("/tmp/spamdb", "w");
@@ -387,9 +392,9 @@ if (typeof getURL == 'undefined') {
 		switch($pkgdb_split[0]) {
 			case "SPAMTRAP":
 				$recordtype = htmlentities($pkgdb_split[0]);
-				$srcip = htmlentities($pkgdb_split[1]);
-				$fromaddress = htmlentities($pkgdb_split[3]);
-				$toaddress = htmlentities($pkgdb_split[4]);
+				$srcip = "";
+				$fromaddress = htmlentities($pkgdb_split[1]);
+				$toaddress = "";
 				$first = "";
 				$pass = "";
 				$expire = "";
@@ -402,7 +407,7 @@ if (typeof getURL == 'undefined') {
 				$toaddress = "";
 				$first = "";
 				$pass = "";
-				$expire = htmlentities($pkgdb_split[2]);
+				$expire = formatspamddatetime($pkgdb_split[2]);
 				$attempts = "";
 				break;
 			case "GREY":
@@ -410,9 +415,9 @@ if (typeof getURL == 'undefined') {
 				$srcip = htmlentities($pkgdb_split[1]);
 				$fromaddress = htmlentities($pkgdb_split[3]);
 				$toaddress = htmlentities($pkgdb_split[4]);
-				$first = htmlentities($pkgdb_split[5]);
-				$pass = htmlentities($pkgdb_split[6]);
-				$expire = htmlentities($pkgdb_split[7]);
+				$first = formatspamddatetime($pkgdb_split[5]);
+				$pass = formatspamddatetime($pkgdb_split[6]);
+				$expire = formatspamddatetime($pkgdb_split[7]);
 				$attempts = htmlentities($pkgdb_split[8]);			
 				break;
 			case "WHITE":
@@ -420,9 +425,9 @@ if (typeof getURL == 'undefined') {
 				$srcip = htmlentities($pkgdb_split[1]);
 				$fromaddress = "";
 				$toaddress = "";
-				$first = htmlentities($pkgdb_split[4]);
-				$pass = htmlentities($pkgdb_split[5]);
-				$expire = htmlentities($pkgdb_split[6]);
+				$first = formatspamddatetime($pkgdb_split[4]);
+				$pass = formatspamddatetime($pkgdb_split[5]);
+				$expire = formatspamddatetime($pkgdb_split[6]);
 				$attempts = htmlentities($pkgdb_split[8]);			
 				break;
 		}
@@ -433,9 +438,9 @@ if (typeof getURL == 'undefined') {
 		echo "<td class=\"listr\">{$srcip}</td>";
 		echo "<td class=\"listr\">{$fromaddress}</td>";		
 		echo "<td class=\"listr\">{$toaddress}</td>";
-		echo "<td class=\"listr\"><span style='white-space: nowrap;'>" . date("Y-m-d", $first) . "<br/>" . date("H:i:s", $first) . "</span></td>";
-		echo "<td class=\"listr\"><span style='white-space: nowrap;'>" . date("Y-m-d", $pass) . "<br/>" . date("H:i:s", $pass) . "</span></td>";
-		echo "<td class=\"listr\"><span style='white-space: nowrap;'>" . date("Y-m-d", $expire) . "<br/>" . date("H:i:s", $expire) . "</span></td>";
+		echo "<td class=\"listr\"><span style='white-space: nowrap;'>" . $first . "</span></td>";
+		echo "<td class=\"listr\"><span style='white-space: nowrap;'>" . $pass . "</span></td>";
+		echo "<td class=\"listr\"><span style='white-space: nowrap;'>" . $expire . "</span></td>";
 		echo "<td class=\"listr\">{$attempts}</td>";
 		echo "<td>";
 		$rowtext = "<NOBR><a href='javascript:toggle_on(\"w{$rows}\", \"/themes/{$g['theme']}/images/icons/icon_plus_p.gif\"); getURL(\"spamd_db.php?buttonid=w{$rows}&srcip={$srcip}&action=whitelist\", outputrule);'><img title=\"Add to whitelist\" name='w{$rows}' id='w{$rows}' border=\"0\" alt=\"Add to whitelist\" src=\"/themes/{$g['theme']}/images/icons/icon_plus.gif\"></a> ";
diff --git a/config/spamd/spamd_settings.xml b/config/spamd/spamd_settings.xml
index 3fdab415..225ef4a6 100644
--- a/config/spamd/spamd_settings.xml
+++ b/config/spamd/spamd_settings.xml
@@ -83,6 +83,16 @@
          will be parsed and rendered for the user as a gui with input, and selectboxes. -->
 	<fields>
 		<field>
+			<fielddescr>Intercept on interfaces</fielddescr>
+			<fieldname>spamdbinds_array</fieldname>
+			<description>
+				<![CDATA[These are the interfaces spamd will intercept smtp connections on.]]>
+			</description>
+			<type>interfaces_selection</type>
+			<multiple>true</multiple>
+			<hideinterfaceregex>loopback</hideinterfaceregex>
+		</field>
+		<field>
 			<fielddescr>Identifier</fielddescr>
 			<fieldname>identifier</fieldname>
 			<description>The SMTP version banner that is reported upon initial connection.</description>
diff --git a/pkg_config.10.xml b/pkg_config.10.xml
index 01082486..a28082e2 100644
--- a/pkg_config.10.xml
+++ b/pkg_config.10.xml
@@ -448,7 +448,7 @@
 		<category>Services</category>
 		<config_file>https://packages.pfsense.org/packages/config/spamd/spamd.xml</config_file>
 		<depends_on_package_pbi>spamd-4.9.1_2-##ARCH##.pbi</depends_on_package_pbi>
-		<version>1.1.2</version>
+		<version>1.1.3</version>
 		<status>Beta</status>
 		<required_version>2.2</required_version>
 		<port_category>mail</port_category>
diff --git a/pkg_config.8.xml b/pkg_config.8.xml
index b45bff1d..38f235f6 100644
--- a/pkg_config.8.xml
+++ b/pkg_config.8.xml
@@ -534,7 +534,7 @@
 		<depends_on_package_base_url>https://files.pfsense.org/packages/8/All/</depends_on_package_base_url>
 		<depends_on_package>spamd-4.9.1.tbz</depends_on_package>
 		<depends_on_package_pbi>spamd-4.9.1-i386.pbi</depends_on_package_pbi>
-		<version>4.9.1 v1.1.2</version>
+		<version>4.9.1 v1.1.3</version>
 		<status>Beta</status>
 		<required_version>1.2.1</required_version>
 		<configurationfile>spamd.xml</configurationfile>
diff --git a/pkg_config.8.xml.amd64 b/pkg_config.8.xml.amd64
index f52f2525..d51dba1d 100644
--- a/pkg_config.8.xml.amd64
+++ b/pkg_config.8.xml.amd64
@@ -521,7 +521,7 @@
 		<depends_on_package_base_url>https://files.pfsense.org/packages/amd64/8/All/</depends_on_package_base_url>
 		<depends_on_package>spamd-4.9.1.tbz</depends_on_package>
 		<depends_on_package_pbi>spamd-4.9.1-amd64.pbi</depends_on_package_pbi>
-		<version>4.9.1 v1.1.2</version>
+		<version>4.9.1 v1.1.3</version>
 		<status>Beta</status>
 		<required_version>1.2.1</required_version>
 		<configurationfile>spamd.xml</configurationfile>