Discussion :

[lodacom]

Bonjour à tous et à toutes,
J'ai codé un script shell que j'ai nommé envoi_automatique_mail.sh que voici:

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
29
30
 
#!/bin/bash
 
cd mails/default
for fich in `ls`
do
    let nbre_fich=$nbre_fich+1
done
if [[ $nbre_fich -eq 0 ]]
then
    echo "Plus de mail à envoyer"
    exit
else
    echo "Nbre de mail à envoyer: $nbre_fich"
    cd /var/www/Penelope
    while [[ $nbre_fich -gt 0 ]]
    do
        let nbre_fich=0
        cd /var/www/Penelope
        php app/console swiftmailer:spool:send --message-limit=1 --env=dev #commande Symfony 2 pour envoyer des mails depuis le spool
        cd /home/lolo/Bureau/Automatic_Tasks/mails/default
        for fich in `ls`
        do
            let nbre_fich=$nbre_fich+1
        done
        echo "Nbre de mail à envoyer: $nbre_fich"
        sleep 20
    done
    echo "Plus de mail à envoyer"
fi

Quand je l'exécute comme ceci:

Code :

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

1
2
 
./envoi_automatique_mail.sh

Depuis mon terminal ça n'envoie pas mes mails. Par contre quand je fait ceci:

Code :

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

1
2
 
sudo ./envoi_automatique_mail.sh

Les envois se passe correctement. Je me suis donc dis, vu que comme je doit faire ça via cron il faut que j'utilise mon utilisateur sudo pour que les envois soient bien fait. Voici la ligne cron que j'ai mise directement dans /etc/crontab:

Code :

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

1
2
 
0  *    * * 1-6 lolo    /home/lolo/Bureau/Automatic_Tasks/envoi_automatique_mail.sh >> /home/lolo/Bureau/Automatic_Tasks/log/log.txt

Le script se lance bien toutes les heures. Mais la ligne d'envoi:

Code :

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

1
2
 
php app/console swiftmailer:spool:send --message-limit=1 --env=dev

n'est pas exécutée et du coup ben ça provoque une boucle infinie puisqu'un mail est stocké sous forme d'un fichier. J'ai l'impression qu'il faut obliger le script à l'exécuter en tant que sudo sinon ça n'enverra jamais rien. Le hic c'est que je ne sais pas le faire. J'ai essayé de chercher sur le net avec plus ou moins de succés (cf les deux liens ci-dessous):

http://www.trustonme.net/didactels/214.html
http://ubuntu.5.x6.nabble.com/su-dan...-td605788.html

Mais j'avoue que je n'ai pas trop compris la soluce . Si quelqu'un pouvait m'expliquer comment faire et/ou me donner une solution plus compréhensible, merci d'avance pour vos réponses .
PS: pour ceux qui aimerai savoir à quoi ressemble l'envoi quand cela fonctionne, voici un screen shot.

[N_BaH]

Bonjour,

pouah ! que c'est mal codé.
pourquoi un if, le while qui suit fait le même travail ?
du coup, pourquoi deux boucle for identiques ?
pourquoi recourir à un script php pour envoyer un mail ?

et enfin, l'inévitable «ON N'UTILISE PAS ls DANS UN SCRIPT !!!»


PS: ouah, quel beau bureau ! mais a-ton besoin de le voir ? en entier ?

[lodacom]

Alors petite explication mon if comme tu dit me sert à stoper le script si jamais il n'y aucun mail à envoyer dès le départ. En effet quand je lance le script je ne peux pas savoir si j'ai des mails ou non à envoyer. Donc oui le while ne fait pas le même travail sinon je ne l'aurait pas mis . Il permet de voir s'il reste des mails à envoyer par rapport à la boucle suivante. Euh j'avoue ceci dit que je pourrais virer ma deuxième boucle for et la remplacer par:

Code :

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

1
2
 
let nbre_fich=$nbre_fich-1

qui est plus simple et plus efficace.
Bonne question. Je recours à script php (de Symfony) pour envoyer mes mails justement parce que je fais du spooling. C'est à dire que je n'envoi pas directement mes mails.
Bon sinon j'ai trouvé une soluce mais ça ne fonctionne pas quand c'est lancé depuis cron je ne comprend pas pourquoi .

Code :

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

1
2
 
sudo php app/console swiftmailer:spool:send --message-limit=1 --env=dev < /home/lolo/Bureau/Automatic_Tasks/mon fichier texte

J'ai lancé le script depuis mon terminal (d'ailleurs je n'ai plus besoin de le lancer avec sudo) ça marche nickel. Mais dès que c'est exécuté avec cron ça fait rien WHY ou entout cas ça n'exécute pas la ligne d'envoi?
Et oui je ne suis pas un accros du shell si tu veux savoir, c'est pour ça que c'est codé avec les pieds mais au moins ça fonctionne sur terminal.

[N_BaH]

si le nombre de fichiers est supérieur à zéro, alors la boucle while ne sera pas exécutée, c'est pourquoi je dis que le if n'est pas utile.

[lodacom]

C'est pas faux. Mais je l'avais mis plus pour avoir une trace sur mon log qu'autre chose en fait. Bref ça ne me dis toujours pas pourquoi ça ne fait pas ce que je lui demande quand j'exécute en cron alors que ça le fait très bien sur mon treminal. Et surtout y a-t-il une solution à mon problème? Et si oui laquelle?
Merci encore pour tes lumières.

[N_BaH]

pour améliorer ton script, et ne pas utiliser ls :

Code :

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

for f in ./*; do test -f "$f" && echo "mise en spool de \"$f\""; done

si tu as besoin d'un indicateur du nombre de fichiers, alors utilise un tableau :

Code :

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

1
2
3
4
5
6
#!/bin/bash
 
shopt -s nullglob
tableau=( ./* ) #s'il ne peut y avoir que des fichiers, sinon on reprend la boucle et le test montrés au-dessus, et on alimente le tableau ainsi tableau+=( "$f" )
echo "il y a ${#tableau[@]} mails à envoyer"
for f in "${tableau[@]}"; do echo "mise en spool de \"$f\""; done