--- lib/PXElator/httpd.pm	2009/08/24 22:04:39	284
+++ lib/PXElator/httpd.pm	2009/08/28 19:41:16	327
@@ -26,6 +26,7 @@
 <a href=/server>server</a>
 <a href=/brctl>brctl</a>
 <a href=/ip>ip</a>
+<a href=/nmap>nmap</a>
 <a href=/client>client</a>
 </div>
 
@@ -54,6 +55,7 @@
 use ip;
 use wireshark;
 use syslogd;
+use nmap;
 
 use CouchDB;
 
@@ -206,17 +208,34 @@
 			)
 			;
 	} elsif ( $path =~ m!^/client(?:/$RE{net}{IPv4}{-keep})?! ) {
-		my $ip = $1 || $client->peerhost;
+		my $ip = $1;
 
-		if ( my $new_ip = $param->{change_ip} ) {
-			client::change_ip( $ip, $new_ip );
-			$ip = $new_ip;
+		if ( $param->{action} eq 'remove' ) {
+			client::remove( $param->{change_ip} );
+			print $client redirect("$url/client");
+			return;
+		} elsif ( $param->{action} eq 'change' ) {
+			if ( my $new_ip = client::change_ip( $ip, $param->{change_ip} ) ) {
+				print $client redirect("$url/client#$new_ip");
+				return;
+			}
 		}
 
-		my $ip_short = (split(/\./, $ip, 4))[3];
+ 		if ( ! $ip ) {
+			my $peer_ip = $client->peerhost;
 
-		# if ( $ip ne $server::ip ) -- not flexible enough for tunnel endpoints
-		if ( $ip_short >= $server::ip_from && $ip_short <= $server::ip_to ) {
+			my $netmask = ip::to_int $server::netmask;
+			my $network = ip::to_int $server::ip & $netmask;
+			my ( $from, $to ) = ( $network | $server::ip_from, $network | $server::ip_to );
+			my $ip_int  = ip::to_int $peer_ip;
+
+			# show edit for clients in our dhcp range
+ 			if ( $ip_int >= ( $network | $server::ip_from ) && $ip_int <= ( $network | $server::ip_to ) ) {
+				$ip = $peer_ip;
+			}
+		}
+
+		if ( $ip && $ip ne $server::ip ) {
 			my $hostname = client::conf( $ip, 'hostname' => $param->{hostname} );
 
 			my @table = (
@@ -232,13 +251,18 @@
 					'mac' => format::mac( $mac => 'html' ),
 					'deploy' => html::select( 'deploy', $deploy, config::available )
 				);
-				$deploy = qq|<h2>PXElinux</h2>| . html::pre( config::for_ip( $ip ) );
+				if ( my $pxelinux = config::for_ip( $ip ) ) {
+					$deploy = qq|<h2>PXElinux</h2>| . html::pre( $pxelinux );
+				}
 			}
 
 			print $client ok
 				, qq|<form method=get>|
 				, html::table( 2, @table ),
-				, qq|<input type=submit value=change></form>|
+				, qq|
+					<input type=submit name=action value=change>
+					<input type=submit name=action value=remove style="color: red">
+					</form>|
 				, $deploy
 				;
 
@@ -251,7 +275,8 @@
 				map {
 					my @c = split(/\s+/,$_);
 					if ( $#c == 5 ) {
-						( uc $c[3] => [ $c[0] , $c[5] ] )
+						client::save_ip_mac( $c[0], $c[3] );
+						( uc $c[3] => $c[5] )
 					} else {
 					}
 				} read_file('/proc/net/arp')
@@ -261,31 +286,21 @@
 
 			print $client ok
 				, qq|<h2>Clients on $server::ip</h2>|
-				, html::table( -5,
-					'ip', 'hostname', 'mac', 'deploy', 'arp',
+				, html::table( -6,
+					'ip', 'mac', 'dev', 'hostname', 'deploy', 'conf',
 					map {
 						my $ip = $_;
-						$ip =~ s{^.+/ip/}{};
-						my $mac = client::mac_from_ip $ip;
-						my $arp = $arp->{ $mac };
-						$arp = $arp ? $arp->[1] : '';
-						$arp =~ s{$ip}{};
+						my $conf = client::all_conf( $ip );
+						my $mac = delete $conf->{mac} || '';
 						(
-							qq|<a href=/client/$ip>$ip</a>|
-							, client::conf( $ip, 'hostname' )
+							qq|<a name=$ip href=/client/$ip>$ip</a>|
 							, format::mac( $mac => 'html' )
-							, html::tt( client::conf( $ip, 'deploy' ) )
-							, $arp
+							, $arp->{$mac}
+							, delete $conf->{hostname}
+							, delete $conf->{deploy}
+							, ( %$conf ? html::pre_dump( $conf ) : qq|<a href=/nmap?scan=$ip>nmap</a>| )
 						);
-					}
-					glob("$server::conf/ip/*") 
-				)
-				, qq|<h2>ARP</h2>|
-				, html::table( -3, 'mac', 'dev', 'ip',
-					map {
-						my $c = $arp->{$_};
-						( format::mac( $_ => 'html' ), $c->[1], $c->[0] )
-					} sort keys %$arp
+					} client::all_ips
 				)
 				;
 		}
@@ -298,6 +313,18 @@
 			, join("\n", map { qq|<a href=/ip/$_>$_</a>| } ( qw/link addr route neigh ntable tunnel maddr mroute xfrm/ ))
 			, ip::html( $1 )
 			;
+	} elsif ( $path =~ m{^/nmap} ) {
+		if ( my $scan = $param->{scan} ) {
+			nmap::scan( $scan );
+			print $client redirect("$url/client#$scan");
+		} else {
+			print $client ok, qq|
+				<form method=get>
+				<input type=text name=scan>
+				<input type=submit value=scan>
+				</form>
+			|;
+		}
 	} elsif ( $path =~ m{^/our/(\w+)/(\S+)} ) {
 		eval 'our $' . $1 . ' = ' . $2;
 		warn $@ if $@;