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

[N_BaH]

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.

en général, ce sont des chemins vers des programmes/bibliothèques qui ne sont pas dans le PATH restreint de cron.

[lodacom]

Merci pour les petits conseils pour les scripts. J'essaierai d'améliorer dès que ce punaise d'envoie fonctionnera.
Alors pour ma ligne envoyant mes mails j'ai carrément mis les chemins complet.
Je l'ai fait pour l'appel de php , pour le script de console et pour fichier contenant le password. Je crois que je peux pas faire mieux et le tout en sudo! .

Code :

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

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

Et rien à faire ça veut pas. Alors petite question même si mon php appartient au root, et la console, et mon fichier à l'utilisateur "lolo". Normalement l'invocation de cette ligne devrait fonctionner puisque je suis en sudo non? J'ai même mis chmod 777 à la console pour être sûr de mon coup.

[N_BaH]

mon fichier ? sans protéger l'espace ?

Code :

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

mon\ fichier

[disedorgue]

Bonjour,

On s'y perd un peu dans tes modifications:
-A quoi ressemble ta ligne crontab au final ?
-En tant que qui est-ce exécuté ?

Si on regarde ton BEAU bureau, on voit que tu as du saisir un mot de passe pour sudo, le problème ne viendrait-il pas de là ?

Dans ton fichier crontab, en principe, tu as le droit de positionner des variables, donc la variable PATH.
Il existe même une variable MAILTO qui si elle est configuré enverra par mail le résultat de la commande.

[lodacom]

Ah lol non en réalité ce n'est pas pour mettre le vrai nom du fichier j’emploie ce nom là. Et pour te rassurer le nom du fichier que j'utilise réellement ne comporte aucun espace.

Code :

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

1
2

sudo /usr/bin/php /var/www/Penelope/app/console swiftmailer:spool:send --message-limit=1 --env=dev < /home/lolo/Bureau/Automatic_Tasks/tutu.txt

Là encore tutu.txt est un exemple. Mon fichier a également une extension .txt je ne sais pas si ça joue ou pas. Ce qui paraitrait bizarre.
Donc j'en reviens à ma remarque y a un truc qui coince. Mais où? J'ai tous les droits partout même sur mon fichier texte qui en 766!!!
Je vais devenir fou. C'est un truc à s'arracher les cheveux.

[lodacom]

Eh bien finalement j'ai trouvé une bien meilleur solution que de passer par un script. en plus la solution est bien mieux sécurisée que celle que je voulais faire au départ avec le shell, c'est à dire que je n'utilise pas les droits sudo. Donc pour ceux qui seraient interessés par un envoi de mail après les avoir récupérés dans un spool avec le framework Symfony, voici les étapes qu'il faut suivre:
Etape 1: mettre en place un script php qui sera à modifier en fonction de votre répertoire que vous aurez choisi pour mettre vos mails en spool

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public function tryToSendMailAction(){
        $root_directory=$this->get('kernel')->getRootDir();
        $app_console=$root_directory."/../app/console";
        $mails=shell_exec("ls /home/lolo/Bureau/Automatic_Tasks/mails/default");//à modifier
        //le shell_exec avec ls renvoi NULL quand le dossier est vide (donc pas de mail à envoyer)
        while (!is_null($mails)){
            $resultat_envoi=shell_exec("php ".$app_console." swiftmailer:spool:send --message-limit=1 --env=dev");
            $mails=shell_exec("ls /home/lolo/Bureau/Automatic_Tasks/mails/default");//MAJ et à modifier
            sleep(20);//on évite le blocage de l'anti-spam
        }
        //on sort de la boucle quand tous les mails ont été envoyés
        return $this->redirect("accueil");//à modifier
    }

Etape 2: référencé votre fonction dans le rooting.yml de votre Bundle où vous avez mis le script

Code :

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

1
2
3
4
5
6
 
acme_renfort_send_mail:
    pattern: /send_mail
    defaults: { _controller: AcmeRenfortBundle:RecoveryManager:tryToSendMail }
    requirements: 
      _method: GET

Là aussi si le coeur vous en dis vous pouvez nommer différement pattern, le controller ...
Etape 3: mettre en place un script shell qui appel la page (j'ai appelé le script envoi_automatique_mail.sh avec les droits suivant: 755)

Code :

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

1
2
3
 
#!/bin/sh
curl -s http://localhost/Penelope/web/app_dev.php/send_mail

Il faudra évidemment modifier l'URL en fonction de ce que vous aurez mis dans le pattern à l'étape 2 et en fonction du nom du projet que vous aurez mis.
Etape 4 (et dernière étape): l'automatisation de l'envoi (car c'est ce qui m'interesse le plus)
la modification avec la commande: crontab -e
suffit largement pas besoin d'aller modifier le fichier /etc/crontab que je faisais au départ de cette discussion. Donc voici la ligne que j'ai ajouté au crontab:
0 * * * 1-6 /home/lolo/Bureau/Automatic_Tasks/envoi_automatique_mail.sh > /dev/null
Comme on le voit, les envois s'effectuent toutes les heures du lundi au samedi.
Voilà si vous respectez ces étapes les mails qui étaient gardés dans le spool de symfony seront envoyés toutes les heures. C'est le résultat que je voulais obtenir. J'ai testé la solution que j'ai décrite ci-dessus, ça fonctionne nickel. Et encore une fois plus besoin de traficoter avec les sudo et autres trucs de ce genre pas très sécurisé.
Merci tout de même à tous pour vos différents apports. En espérant que cette solution soit utile pour ceux qui veulent faire la même chose.