Discussion :

[jaraf]

Bonjour, je suis débutant en script et je suis sur un projet personnel
Je veux faire ça sur un script bash qui fait les actions suivants:
1/ Se connecter au site www.monsite.fr avec le compte user@monsite.fr et le mot de passe "passer".
2/ En gardant les cookies précédemment créés, effectuer une recherche sur la page boutique et afficher le nombre de produits disponibles dans la page qui ont un tarif supérieur ou égal à 1000€.
3/ Envoyer ce nombre (le nomre trouvé) par mail à l'adresse adresse@monsite.fr
4/ Gérer les erreurs [log](par exemple si le site ne fonctionne pas) et envoyer un mail avec l'erreur dans ce cas à l'adresse adresse@monsite.fr

Ce que j'ai fait:

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
24
25
26
27
28
 
#!/bin/bash 
echo "ce script 
 
 #site= monsite.fr
 #login=test@monsite.fr
 #password=passer
 
 
 
 
wget -c "http://www.monsite.fr/SuiteLienIdentification" --post-data="PersistentCookie=1&LOGIN=test@monsite.fr&PASSWORD=passer" --no-check-certificate --save-cookies="/tmp/cookies" --keep-session-cookies
 
 #etape_pour_telecharger_une_pageAlpha qui contient 
 wget -c "http://test@monsite.fr/pageAlpha.html" --no-check-certificate --load-cookies="/tmp/Tcookie"
 
product="http://test@monsite.fr/pageAlpha.html"
if product >= 1000
	then echo product="produit superieur ou egal à 1000"
	else echo "produit inferieur à 1000 euros"
fi
#lister les logs
ll /tmp/*.log |while read ErreurSite
do
mail -s "erreur" test@monsite.fr < $ErreurSite
done
 sleep 2 
exit 0

Je sais que ça porte des erreurs mais j'aimerais juste avoir votre avis et des pistes pour l'améliorer sur pour le tri et l'envoie de mail.
Cordialement

[N_BaH]

Bonjour,

n'as-tu pas une machine capable d'exécuter ton script ?

Code :

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

1
2
product="http://test@monsite.fr/pageAlpha.html"
if product >= 1000

comme tu peux le voir, product ne contient pas un nombre, mais l'adresse de la pageAlpha,
et, la syntaxe correcte pour la commande if, c'est if <commande>; then..., or, la simple expression d'une comparaison ne constitue pas une commande shell de test

Code :

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

ll /tmp/*.log |while read ErreurSite

les alias d'un interpréteur interactif (le shell dans lequl tu exécutes le script) ne sont pas disponibles dans un interpréteur non-interactif (un script) (où par défaut, ils sont désactivés).
et, pour lister les fichiers d'un répertoire, on utilise une boucle for avec le Développement des chemins (cf. man bash)

je crois qu'il faut régler ces problèmes avant de tenter d'envoyer un mail.

[jaraf]

Merci N Bah pour votre retour.
Oui je ne suis pas trop en dev script.
Je vois les erreurs que j'ai fait .
Je vais les corriger et voir.
Mais à part celà?
Ma syntaxe pour l'envoie de mail elle est bonne?
Pour récupérer st stocker les cookies?

[Sve@r]

Bonjour

Code bash :

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

1
2
3
4
5
6
7
8
#!/bin/bash 
echo "ce script     # Déjà manque les guillemets fermants
 
...

 #etape_pour_telecharger_une_pageAlpha qui contient 
 wget -c "http://test@monsite.fr/pageAlpha.html" --no-check-certificate --load-cookies="/tmp/Tcookie"

Si tu exécutes wget simplement, alors tu n'auras pas possibilité de récupérer quoi que ce soit. En shell, quand on veut récupérer le retour d'une commande (ou, pour être plus exact, ce qu'elle affiche), faut l'encapsuler par $(...) (ou bien par `...` qui est l'ancienne écriture) et bien entendu si on récupère quelque chose alors c'est pour s'en servir donc on met (au minimum) une variable pour ça
pageAlpha=$(wget -c "http://test@monsite.fr/pageAlpha.html" --no-check-certificate --load-cookies="/tmp/Tcookie").
pageAlpha=`wget -c "http://test@monsite.fr/pageAlpha.html" --no-check-certificate --load-cookies="/tmp/Tcookie"`.
Ensuite à toi de traiter le contenu de "$pageAlpha"...

Code bash :

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

1
2
3
4
5
product="http://test@monsite.fr/pageAlpha.html"
if product >= 1000
	then echo product="produit superieur ou egal à 1000"
	else echo "produit inferieur à 1000 euros"
fi

Ici grosse lacune. Déjà une variable se précise en mettant un "$". De plus, si tu écris var="toto" ben il n'y a aucune raison pour que le shell comprenne que tu veux en fait exécuter la commande "toto" et enfin en shell, on ne peut pas évaluer directement une expression. Le "if" ne peut évaluer que des commandes. D'où la commande "test" qui est faite pour ça.

Code bash :

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

1
2
3
4
product=$(la commande qui va bien)
if test $product -ge 1000; then 
    ...
fi

Ou bien (puisqu'une variable qui n'est utilisée qu'une seule fois ne sert à rien)...

Code bash :

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

1
2
3
if test $(la commande qui va bien) -ge 1000; then 
    ...
fi

Code bash :

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

1
2
#lister les logs
ll /tmp/*.log |while read ErreurSite

Pourquoi faire "ll" (équivalent de "ls -l") ? T'as vraiment besoin des infos du format long telles que les droits et la taille ?? Ou tu veux juste les noms de fichiers ??? Surtout que, comme l'a précisé N_Bah, si "ll" est un alias, alors ça ne fonctionnera même pas.

Code bash :

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

1
2
3
ls /tmp/*.log |while read ErreurSite; do
  ...
done

Mais sinon autant éviter "ls" quand ce n'est pas indispensable...

Code bash :

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

1
2
3
for ErreurSite in /tmp/*.log; do
  ...
done