Discussion :

[K4LIN00B]

Bonjour;
j'ai créée un petit script qui permet d'envoyer une requête Ping à chaque ip contenue dans un fichier texte. Le script me dit ensuite pour chacun Ip si la machine est ONLINE ou OFFLINE.
Et le scan recommence tous les x secondes.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
#!/bin/bash
while true; do
	for i in $( cat $repertoireIP )
	do	
		ping -q -c2 $i > /dev/null
 
		if [ $? -eq 0 ]
		then
			fonction_ONLINE
 
		elif [ $? -eq 1 ]
		then
			fonction_OFFLINE	
		fi	
	done
	sleep $rfch

Avec le code en entier et toutes ces fonctionnalités j'optient ce résultat :



(Le OFFLINE passe bien sur en ONLINE quand la machine ciblé ce connecte à internet. )

Le code n'est certainement pas bien optimisé mais il fonctionne c'est tout ce que je lui demande.
Je voudrais maintenant ajouter une fonctionnalité qui permet d'afficher lors du rapport de scan le nom de la machine devant l'ip de la machine (ce qui est plus simple pour l'identifier rapidement). J'aimerais qu'à partir d'un fichier texte de cette forme :

NOM_MACHINE1
xxx.xxx.x.xx
NOM_MACHINE2
xxx.xxx.x.xx

Le script soit capable de faire la différence entre la ligne qui contient le nom de la machine et la ligne de dessous qui contient lip de la machine. Et d'effectuer une action différente en fonction.

Pour moi il devrait avoir cette forme :
( # = ce que je voudrais obtenir )

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
#!/bin/bash
while true; do
	for i in $( cat $repertoireIP )
	do	
                # si la variabe i contient du texte, alors : 
                echo $i
                #puis recommencer la boucle en lissant la ligne suivante.
 
                # Ou si la variable i contient une adresse ip, alors :
 
		ping -q -c2 $i > /dev/null
 
		if [ $? -eq 0 ]
		then
			fonction_ONLINE
 
		elif [ $? -eq 1 ]
		then
			fonction_OFFLINE	
		fi	
	done
	sleep $rfch

Même avec des recherches sur internet je n'est pas trouvé de fonction capable de faire la différence entre la ligne 1 et la ligne 2 de mon fichier texte.

J'espère avoir été clair dans mes explications, si quelqu'un peut m'aider, même à trouver un début de réponde, un bout de code, je lui en serais reconnaissant

[N_BaH]

Bonjour,

Le code n'est certainement pas bien optimisé mais il fonctionne c'est tout ce que je lui demande.

avec un tel raisonnement, tu vas faire des choses compliquées, alors qu'elles devraient être simples, et finir par te retrouver dans une telle situation que tu devras réécrire tous tes scripts en suivant, cette fois, les bonnes pratiques.

d'abord, Comment lire un fichier en shell.
ensuite, tu ajouteras dans la boucle une commande read, et éventuellement testeras si la ligne lue est paire ou impaire (le nom des machines est sur des lignes impaires) !

[K4LIN00B]

Ouaw ! Déja merci pour ta réponse plus que rapide !

Tu as raison, faut que je fasse un code propre. Mais concrètement je ne pense pas savoir faire du code proprement !

tu ajouteras dans la boucle une commande read, et éventuellement testeras si la ligne lue est paire ou impaire (le nom des machines est sur des lignes impaires) !

Merci pour les liens et cette piste de recherche, cela va pas mal m'aider je pense !

[Flodelarab]

Bonjour

T'enflamme pas. Il s'agit d'une modification de fichier texte. On peut conseiller aussi d'utiliser grep/sed/awk/etc...

Peux-tu donner un exemple de fichier en entrée et un exemple de fichier en sortie (souhaité) ?

[BufferBob]

salut,

allez je m'lance :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
while true; do
   while read name addr; do {
      ping -c1 -W1 $addr &>/dev/null
      case $? in
         0) echo "$name is ONLINE";;
         1) echo "$name is OFFLINE";;
         *) echo "PING returned an error code (machine=$name, ip=$addr)";;
      esac
   } & done < <(paste -d' ' - - < "$repertoireIP") | sort
   sleep $rfch
done

• pour mettre en forme le fichier au départ je lui met un coup de paste, ce qui permet de lire à chaque boucle le nom de la machine et l'IP qui lui correspond
• on remplace le if/elif/then par un case/esac qui permettra de prendre en compte les cas où ping tombe en erreur
• les ping sont lancés en parallèle {...} &, ce qui nous fait gagner du temps

pour tout le reste, il y a fping

[K4LIN00B]

Merci pour vos réponses ! En venant sur ce forum je pensais pas obtenir de l'aide aussi rapidement.

allez je m'lance :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
while true; do
   while read name addr; do {
      ping -c1 -W1 $addr &>/dev/null
      case $? in
         0) echo "$name is ONLINE";;
         1) echo "$name is OFFLINE";;
         *) echo "PING returned an error code (machine=$name, ip=$addr)";;
      esac
   } & done < <(paste -d' ' - - < "$repertoireIP") | sort
   sleep $rfch
done

7h du matin, t'etais motivé

case/esac est exactement ce qu'il me fallait ! Je voulais absolument caler mon if/elife/else et ce n'était pas la bonne solution.
Je ne savait pas qu'on pouvais lancer en parallèle avec {...} & et je ne connaissais pas non plus paste -d, ces fonctions sont ultra-utile dans mon cas.

Par contre 'ai pas compris l'utilité de sort dans le code. Et j'ai une erreur à la ligne 10 : Syntax error: redirection unexpected
Je ne serais pas dire à quoi cela est dû...

[disedorgue]

Bonjour,

Je pense que tu n'as pas respecté l'espace entre la redirection de fichier et la redirection vers le processus:
Ligne bonne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

} & done < <(paste -d' ' - - < "$repertoireIP") | sort

Ligne fausse:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

} & done <<(paste -d' ' - - < "$repertoireIP") | sort

[K4LIN00B]

Bonjour,

l'espace est bien respecté... Le problème ne peut pas venir d'ailleurs ?

[N_BaH]

le script me rend une erreur pour la ligne 10 : Syntax error: redirection unexpected

sûrement parce que tu fais exécuter le script par sh leScript (alors qu'il s'agit d'une syntaxe bash), au lieu de l'avoir rendu exécutable (chmod +x leScript), et de l'appeler simplement avec son chemin :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
cd /chemin/ou/est/
./leScript

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

/chemin/ou/est/leScript

en ayant pris soin de conserver le shebang #!/bin/bash

[disedorgue]

Si, tu n'es pas en bash (ne fonctionne pas par exemple en dash)

[K4LIN00B]

Quel couillon je suis
J'ai pris pour (mauvaise ?) habitude lancer tous mes scripts avec la commande :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
cd /repertoire_script
sh nom_du_script.sh

Alors qu'avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

/chemin/ou/est/leScript

et en l'ayant rendu executable cela fonction niquel !

Mes codes couleurs ne fonctionnent plus mais je pense savoir comment régler cela (puis c'est que de l'esthétique après tout)

Milles merci à vous