[BASH] Probleme Difference

Probleme Difference [BASH] - Shell/Batch - Programmation

Marsh Posté le 03-03-2014 à 14:55:12    

Bonjour tout le monde,

 

je me bagarre avec mon script Shell depuis quelques temps.

 

Son but :

 

Récupérer mon IP publique, puis la stocker dans un fichier ip_temp.

 

Ensuite si le contenu du ip_temp est différent du fichier de réference monippublique.txt, il devra injecter la nouvelle ip dans le fichier de ref et ensuite m'envoyer un mail avec ma nouvelle IP.

 

Ce script sera executé en tâche de fond via Cron

 
Code :
  1. #!/bin/bash
  4. wget http://checkip.dyndns.org/ -O - -o /dev/null | cut -d: -f 2 | cut -d\< -f 1 > /tmp/ip_temp
  6. #VARIABLES
  8. T1=`cat /tmp/ip_temp`
  9. T2=`cat /tmp/monippublique.txt`
  11. #Check Dev
  13. echo $T1
  14. echo $T2
  16. #Comparaison , maj et envoi
  18. if [ "$T1" != "$T2" ]; #j'ai essayé sans quote et avec
  19.     then
  20. #       echo T1 > /tmp/monippublique.txt
  21. #       echo "l'ip a change et est devenue $T1" |mail -s "changement" mon-email@gmail.com
  22.         echo "c'est bon"
  23. fi
 

CODE : code provisoire le temps de régler le pb

 


Mode debug

 
Code :
  1. + wget http://checkip.dyndns.org/ -O - -o /dev/null
  2. + cut -d: -f 2
  3. + cut '-d<' -f 1
  5. ++ cat /tmp/ip_temp
  6. + T1=' 111.11.111.1'
  8. ++ cat /tmp/monippublique.txt
  9. + T2=' 111dsdsd.11.111.1'
  11. + echo 111.11.111.1
  12. 111.11.111.1
  14. check_ip_publique.sh: line 27: 111.11.111.1: command not found
  16. + echo 111dsdsd.11.111.1
  17. 111dsdsd.11.111.1
  19. + ["$T1"!="$T2"]
  21. check_ip_publique.sh: line 33: [$T1!=$T2]: command not found
 

mon fichier de ref a volontairement des lettres dans son IP pour que la MAJ se fasse


Message édité par nomiss le 03-03-2014 à 15:49:05
Reply

Marsh Posté le 03-03-2014 à 14:55:12   

Reply

Marsh Posté le 03-03-2014 à 18:18:23    

Es-tu sur de contenu du fichier /tmp/ip_temp ? Il n'y a pas de ligne blanche ou autre ? Vu que tu le génère à partir d'un site que tu ne maîtrise pas toi, en ce qui me concerne je mettrais un certain nombre de protection, notamment je vérifierais via regexp que j'obtiens bien une et une seule adresse IP sans rien d'autre. D'ailleurs il ne sert a rien de passer par un fichier temporaire non plus. En gros je ferais quelque chose comme (fait vite fait sans test il y a surement mieux a faire) :
 

Code :
  1. T1=`wget http://checkip.dyndns.org/ -O - -o /dev/null | cut -d: -f 2 | cut -d\< -f 1 | grep -E "^[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}$" | tail -1`
  2. [ -z "$T1" ] && exit 1


 
Le mode debug que tu montres ne corresponds pas à ton code :
Quelle est la ligne 27 de ton script ? c'est comme si tu avait fait "$T1" au lieu de "echo $T1".
 
Le problème de la ligne 33 est le manque d'espace entre les "[" et les valeurs.
Il faut bien des quotes également. Si tu veux oter tout doute lié à la présence de variables vide par exemple tu peux modifier le test pour faire quelque chose comme :

Code :
  1. [ "x${T1}" != "x${T2}" ]


 
Attention dans la ligne en commentaire "echo T1 > /tmp/monippublique.txt" il n'y a pas de $ avant le nom de la variable.

Reply

Marsh Posté le 04-03-2014 à 08:54:39    

Salut, merci pour ta réponse,  
 
- "D'ailleurs il ne sert a rien de passer par un fichier temporaire non plus"
 
en fait à la base j'avias fais ça :
 
T1=`wget http://checkip.dyndns.org/ -O - -o /dev/null | cut -d: -f 2 | cut -d\< -f 1 > /tmp/monippublique.txt`
 
je vais vérifier tout ça ...
 
[edit]
 
- "Es-tu sur de contenu du fichier /tmp/ip_temp ? Il n'y a pas de ligne blanche ou autre ?"
 
via le lien checkip, il me retourne l'@ avec un espace au début.
 
- "En gros je ferais quelque chose comme (fait vite fait sans test il y a surement mieux a faire) :"
 
J'ai testé :
 

Code :
  1. T1=`wget http://checkip.dyndns.org/ -O - -o /dev/null | cut -d: -f 2 | cut -d\< -f 1 | grep -E "^[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}$" | tail -1`

 
 
il ne me retourne rien.
 
- "Quelle est la ligne 27 de ton script ? c'est comme si tu avait fait "$T1" au lieu de "echo $T1"."
 

Code :
  1. #!/bin/bash
  3. echo Execution du script
  4. echo
  5. echo Adresse Publique Actuelle :
  9. wget http://checkip.dyndns.org/ -O - -o /dev/null | cut -d: -f 2 | cut -d\< -f 1 > /tmp/ip_temp
  12. #VARIABLES
  14. T1=`cat /tmp/ip_temp`
  15. T2=`cat /tmp/monippublique.txt`
  17. #Check
  19. echo $T1
  21. $T1
  23. echo $T2
  25. #Boucle de verification
  27. if ["$T1" -ne "$T2"]                         (ndlr : pour trouvé mon problème j'ai fias un script à côté avec ce type de notation fonctionnelle)
  28.     then
  29. #       echo T1 > /tmp/monippublique.txt
  30. #       echo "l'ip a change et est devenu $T1" |mail -s "changement" monemail@gmail.com
  31.         echo "super"
  32. fi
  37. echo Termine !


 

Code :
  1. [root@pocpup tmp]# bash -x check_ip_publique.sh
  2. + echo Execution du script
  3. Execution du script
  4. + echo
  6. + echo Adresse Publique Actuelle :
  7. Adresse Publique Actuelle :
  8. + wget http://checkip.dyndns.org/ -O - -o /dev/null
  9. + cut -d: -f 2
  10. + cut '-d<' -f 1
  11. ++ cat /tmp/ip_temp
  12. + T1=' 999.11.254.88'
  13. ++ cat /tmp/monippublique.txt
  14. + T2=' 999dsdsd.11.254.88'
  15. + echo 999.11.254.88
  16. 999.11.254.88
  17. + 999.11.254.88
  18. check_ip_publique.sh: line 21: 999.11.254.88: command not found
  19. + echo 999dsdsd.11.254.88
  20. 999dsdsd.11.254.88
  21. + '[ 999.11.254.88' -ne ' 999dsdsd.11.254.88]'
  22. check_ip_publique.sh: line 27: [ 999.11.254.88: command not found
  23. + echo Termine '!'
  24. Termine !
  26. (ndlr : Biensûr l' "ip" est volontairement changée)

Message cité 1 fois
Message édité par nomiss le 04-03-2014 à 09:13:00
Reply

Marsh Posté le 04-03-2014 à 11:56:22    

nomiss a écrit :

en fait à la base j'avias fais ça :

 

T1=`wget http://checkip.dyndns.org/ -O - -o /dev/null | cut -d: -f 2 | cut -d\< -f 1 > /tmp/monippublique.txt`

 

je vais vérifier tout ça ...

 

Normal que ca ne donne rien dans T1, si tu mets une redirection (le > ) tu n'auras rien dans la variable T1

 
nomiss a écrit :


via le lien checkip, il me retourne l'@ avec un espace au début.

 

J'ai testé :

 
Code :
  1. T1=`wget http://checkip.dyndns.org/ -O - -o /dev/null | cut -d: -f 2 | cut -d\< -f 1 | grep -E "^[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}$" | tail -1`

 

il ne me retourne rien.

 

Normal, si il y a un espace avant, il faut l'inclure dans la regexp, ou alors le supprimer avant de passer la regexp

 


nomiss a écrit :


echo $T1

 

$T1

 



Le $T1 tout seul la ne sert a rien. C'est lui qui te fait une erreur de syntaxe : le système essaie d'exécuter le contenu de la variable, donc ton IP, ce qui n'est pas une commande valide

 
nomiss a écrit :


if ["$T1" -ne "$T2"]


Il manque les espaces apres et avant les [ ]. En outre le -ne ne sert qu'a tester des entiers, ce qui n'est pas le cas en l'occurence. Il faut donc utiliser !=

 
Code :
  1. if [ "$T1" != "$T2" ]


Message édité par Nukolau le 04-03-2014 à 11:58:20
Reply

Marsh Posté le 04-03-2014 à 11:58:23    

Code :
  1. #!/bin/bash
  3. echo Execution du script
  4. echo
  5. echo Adresse Publique Actuelle :
  9. wget http://checkip.dyndns.org/ -O - -o /dev/null | cut -d: -f 2 | cut -d\< -f 1 > /tmp/ip_temp
  12. #VARIABLES
  14. T1=`cat /tmp/ip_temp`
  15. T2=`cat /tmp/monippublique.txt`
  16. #T1=A
  17. #T2=B
  19. #Check
  21. echo $T1
  23. echo $T2
  25. #Boucle de verification
  27. if [ " ${T1}" != " ${T2}" ]
  28.     then
  29.         echo T1 > /tmp/monippublique.txt
  30.         echo "l'ip a change et est devenu $T1" |mail -s "changement" monemail@gmail.com
  31.         echo "super"
  32. fi
  37. echo Termine !
 


je n'ai plus d'erreur, par contre j'attend encore mon mail.

 

Je vais aussi modifier T1 et mettre lcommande wget dedans

 

[edit] j'ai pris note de tout ce que tu m'as dis, merci.


Message édité par nomiss le 04-03-2014 à 13:27:02
Reply

Marsh Posté le 05-03-2014 à 19:50:39    

Script Opérationnel !  
 

Code :
  1. #!/bin/bash
  4. #VARIABLES
  6. T1=`wget http://checkip.dyndns.org/ -O - -o /dev/null | cut -d ' ' -f 6 | cut -d\< -f 1`
  7. T2=`cat /home/ScriptIP/IP`
  9. #Boucle de verification
  11. if [ "$T1" != "$T2" ] #Si le resultat du retour commande est different du fichier de ref
  12.     then
  13.         echo $T1 > /home/ScriptIP/IP    #MISE A JOUR DU FICHIER IP
  14.         echo "Voici la nouvelle IP Publique => $T1 " |mail -s "Nouvelle IP Publique" monmail@gmail.com #ENVOI DU MAIL AVEC LA NOUVELLE IP PUBLIQUE
  15. fi
  17. #FIN


 
à présent je vais le mettre en Cron,
 
Comment ça se passe pour ça ? (CENTOS 6.5)
 
L'idéal serait des le boot de ma VM et puis toute les 10mn qui suivent, possible ?

Reply

Marsh Posté le 06-03-2014 à 22:09:50    

Un cron est exécuté à intervalles de temps. Pour exécuter un programme au boot, il te faut un script de boot.
http://www.commentcamarche.net/faq [...] it-et-cron

Reply

Marsh Posté le 07-03-2014 à 10:23:53    

Oui j'avais pensé à ça. Je vais exploiter ce lien, merci
 
[edit]
 
j'ai juste à mettre mon .sh dans le init.d ? et de parametrer ensuite un cron ?


Message édité par nomiss le 07-03-2014 à 10:25:57
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed