SAUVEGARDE SQL ET DOSSIERS

Commentaire de yoman64 le 09/03/2008 01:10:26

Salut,
J'ai jetté un rapide coup d'oeil à ton code et j'ai quelques commentaires.

Tout d'abord tu n'échappe pas les données passé à mysqldump, si mon mot de passe contient des chars spéciaux tel que | ou ; ou " ton script va lamentablement échouer; renseigne toi sur la commande escapeshellarg. Ensuite dans config.php $base_number=3; est totalement inutile, renseigne toi sur la commande count.

Dernièrement tu devrais passer la sortie de mysqldump à gzip au lieu de le piper dans un fichier, ça éviterais des IO inutiles.
Dernièrement extract est très dangereux, surtout de la façon que tu l'utilise, et tu peux très bien t'en passer :)

Et pour finir préfère l'utilisation des ' plutot que des " .


Bonne chance :)

Commentaire de yoman64 le 09/03/2008 02:11:09

Bah tu pourrais remplacer
"mysqldump --host=$host --user=$user --password=$pwd $base | gzip > $filename"
par 'mysqldump --host='.$host.' --user='.$user.' --password='.$pwd.' '.$base.' | gzip > '.$filename'


MAIS ceci dit je suis d'accord que les " " dans ce cas ci sont beaucoup plus pratiques, ils sont là pour ça après tout :) Ma remarque concernait surtout les simples strings qui n'ont pas besoin d'être parsés notamment dans le fichier de config , bref je crois que tu as bien compris où je voulais en venir :)