[Bash] Scripts vsftpd

[Ekroz]

Bonjour à toute la communauté N-PN !
Cela fait un moment que l'on ne me voit plus trop, rassurez-vous je n'ai toujours pas été kidnappé par les Illuminatis.
Je suis actuellement en stage dans une PME et j'ai pour mission de créer un serveur FTP (j'ai choisi vsftpd pour sa sécurité).

J'ai remarqué que la création, la modification et la suppression d'utilisateurs virtuels "à la main" était un travail de longue haleine lorsqu'il s'agit de gérer plusieurs dizaines de comptes.

J'ai donc codé 4 scripts Bash afin d'automatiser :

• La création d'utilisateurs : http://pastebin.com/mjWu2XtE
  
• La récupération de mot de passe : http://pastebin.com/rLyig3Db
  
• La modification de mot de passe : http://pastebin.com/et0wGu4F
  
• La suppression d'utilisateurs : http://pastebin.com/dE6T40D9
  

Je partage donc le code source si jamais vous êtes confronté un jour à ce cas, libre à vous donc de le réutiliser, le modifier et le partager.

N'étant pas un 1337 h4xx0r du Bash, n'hésitez pas à faire des remarques concernant les scripts, me proposer des améliorations, le but premier étant de faire quelque chose de sécurisé, rapide et simple.

Amicalement,
Ekroz.

[Junky]

Bonjour,

Merci du partage.

Sans vouloir aller chercher la p'tit bête, tu devrais imo passer par des variables et éviter de mettre en dur les path des files. Si un jour pour une question ou une autre tu dois changer de rep, tes scripts deviennent pénibles a modifier. Tu peux faire un truc du genre:

Code BASH :


conf_dir="/etc/vsftpd/"
login_file="login.txt"
backup_ext=".bak"

ect....
 

Si un jour, tu changes de repertoire, tu aura besoin de modifier QUE les variables et non tout le programme.

En plus en mettant en place cette dynamique, ca te permet de faire une fonction de check. Plein de fois dans ton code tu vas checker si le backup d'un fichier c'est bien réaliser. Donc a chaque fois tu poses un:

Code BASH :


if [ ! f CONDITION]; then
     blablabla....
fi
 

Dans le premier script tu en as 4 ou 5.. N'oublie pas qu'en dev a partir du moment ou tu fais 2 fois la même chose, il faut faire une fonction.

Donc un truc du genre:

Code BASH :


check() {
if [ ! f $conf_dir$login_file$backup_ext ]; then
        echo "The game"
        exit 1
fi
}
 

Après dans l'ensemble si ca fait le taf j'ai envie de dire GG. Mais regarde un peu plus sur l'optimisation de ton code...

Ah oui dans le 4ème script, mis WTF le if... Il déroule sur 4km le truc... Je l'ai survolé seulement (j'suis au taf pas eu le tps de le lire correctement) mais en revenant sur mon idée précédente de fonction, il y a surement de faire un truc vachement plus propre...

Bon courage.

Junky

[gruik]

[thxer]

Merci à vous trois .

[Ekroz]

Merci beaucoup pour vos réponses, j'ai pu mettre à jour mes scripts selon vos conseils.
Les liens Pastebin ont été mis à jour et pour ceux qui ont la flemme de remonter en haut du topic, les voici :

• La création d'utilisateurs : http://pastebin.com/mjWu2XtE
  
• La récupération de mot de passe : http://pastebin.com/rLyig3Db
  
• La modification de mot de passe : http://pastebin.com/et0wGu4F
  
• La suppression d'utilisateurs : http://pastebin.com/dE6T40D9