wan ip check script

[marc heijerman]

#!/bin/bash

ip=`wget -q -O - http://ip.keithscode.com`
file='/home/marc/scripts/files/wanip.txt'
file2='/home/marc/scripts/files/ipchanged.txt'

test1=`cat $file | awk "NR==1"`
test2=`cat $file | awk "NR==2"`

recipient='name@domain'

touch $file2
test -x $file

if [ "$?" != 0 ]
 then
  touch $file
  echo $ip > $file
fi

echo $ip >> $file

if [ "$test2" != "$test1" ]
 then
   echo "ISP changed your dhcp ip address from:$test1 to:$test2" > $file2
   mailx -s "IP Change" $recipient < $file2
else
   sed -i 2d $file
   echo > $file2
fi





Nu werkt dit nog niet optimaal. Wanneer ik het script laat lopen, dan werkt de eerste keer het sed -i 2d command niet, er blijven dus 2 ip's in $file staan. De tweede keer, wordt het tweede ip wel verwijderd enz...

Ik doe hier iets niet goed, alleen zie ik even niet meer wat ik niet goed doe.

Iemand een ideetje ?
exit

[grizzler]

Wat probeer je precies te doen? Alleen maar een waarschuwing te geven als het IP-adres is veranderd? Waarom sla je het dan twee keer op? Zoals het er nu bij staat, heb je beurtelings één en twee regels in wanip.txt, zodat je bij gelijkblijvend IP-adres beurtelings wel en geen match krijgt bij de laatste test.
Zou dit niet voldoende zijn?

#!/bin/bash

ip=`wget -q -O - http://ip.keithscode.com`
file='/home/marc/scripts/files/wanip.txt'
file2='/home/marc/scripts/files/ipchanged.txt'

recipient='name@domain'

if [ ! -e $file ]; then
  echo $ip > $file
fi

test1=`cat $file`

if [ "$test1" != "$ip" ]; then
  echo "ISP changed your dhcp ip address from:$test1 to:$ip" > $file2
  mailx -s "IP Change" $recipient < $file2
  echo $ip > $file
fi

[marc heijerman]

Dat ziet er inderdaad wat beter uit.....

Moet nog een beetje inkomen in het scripten

Ik was inderdaag wat dubbel en omslachtig aan het doen , ziet er wat overzichtelijker en simpeler uit zo....

Thanks!