Exercice Perso en Shell

Bonjour/bonsoir à tous et toutes,

je viens vers vous car j'ai un petit exo perso à faire, qui consiste à cela :

J'ai un fichier csv qui est composé de :
Identifiant,Nom,Prenom,Service,Fonction,Permanent,
info1,Napo,Léon,info,Directeur,Oui,
info2,Scripte,Perle,info,Responsable,Oui,
info3,Scrout,Jessica,info,Employe,Oui,
info4,Martin,Henri,info,Employe,Non,
compta1,Terrieur,Alex,compta,Directeur,Oui,
compta2,Eràk,Patrique,compta,Responsable,Oui,
compta3,Personne,Jonathan,compta,Employe,Non,
compta4,Demùsset,Alfred,Compta,Employe,Non,
boss,MANVUSSA,Gérard,direction,Directeur,Oui,
info5,Terrieur,Alain,info,Employe,Non,
info4,Martin,Henri,info,Employe,Non,
comm1,MANVUSSA,Gérard,commercial,Directeur,Oui,
comm2,Despoireaux,Yvan,commercial,Employe,Oui,
comm3,Trëchere,Yvan,commercial,Employe,Non,

et je dois faire cela :

Pour chaque service, créer un répertoire commun à l’ensemble des personnes du service.
Dans le répertoire du service créer pour chaque utilisateur du service un répertoire personnel.
Créer les utilisateurs en respectant les obligations suivantes :
Pour les CDI (permanent) affecter un mot de passe et un login en mode shadow.
Pour les CDD définir un compte provisoire pour 60 jours.


du coup pour l'instant j'ai écris cela :

Code:

---

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 31 32 33 34 35 36 37 38 39 40 41

#!/bin/sh date_expire=`date -d "60 days" +%Y-%m-%d` while IFS=',' read Identifiant Nom Prenom Service Fonction Permanent do line="$Identifiant $Nom $Prenom $Service $Fonction $Permanent"   if grep -q "$Service" "/etc/group" ;then echo "$Service Trouvé" else echo "creation du Groupe $Service" groupadd $Service; fi   if [ ! -d "/home/$Service" ];then echo "Création du répertoire $Service"; mkdir /home/$Service chown -R root:$Service /home/$Service fi   if [ ! -d "/home/$Service/$Prenom$Nom" ];then echo "Création Répertoire $Prenom$Nom"; mkdir /home/$Service/$Prenom$Nom chown -R root:$Service /home/$Service/$Prenom$Nom fi   ##generation du password en curl ## #pwd=`curl --silent http://www.sethcardoza.com/api/rest/tools/random_password_generator` #echo "$Permanent" if [ $Permanent = "oui" ];then #echo "$Permanent" pwd=`curl --silent http://www.sethcardoza.com/api/rest/tools/random_password_generator` useradd "$Identifiant" -p "$pwd" -d "/home/$Service/$Prenom$Nom" -e "" -g "$Service" chown -R "$Identifiant":"$Service" "/home/$Service/$Prenom$Nom" else useradd "$Identifiant" -p "$pwd" -d "/home/$Service/$Prenom$Nom" -e "$date_expire" -g "$Service" chown -R "$Identifiant":"$Service" "/home/$Service/$Prenom$Nom" fi       done < /home/toto/fichier-edit.csv

---

J'arrive bien à créer les différents répertoires pour les services :

Code:

---

1 2 3 4 5 6 7

drwxr-xr-x 24 root root 4096 déc. 14 11:26 .. drwxr-xr-x 5 root commercial 4096 déc. 15 08:43 commercial drwxr-xr-x 5 root compta 4096 déc. 15 08:42 compta drwxr-xr-x 3 root Compta 4096 déc. 15 08:42 Compta drwxr-xr-x 3 root direction 4096 déc. 15 08:43 direction drwxr-xr-x 7 root info 4096 déc. 15 08:43 info drwxr-xr-x 3 root Service 4096 déc. 15 08:42 Service

---

ainsi que les répertoires de chaques utilisateurs dans le service correspondant (exemple ici pour le service "compta"):

Code:

---

1 2 3 4

drwxr-xr-x 2 compta1 compta 4096 déc. 15 08:42 AlexTerrieur drwxr-xr-x 2 compta3 compta 4096 déc. 15 08:42 JonathanPersonne drwxr-xr-x 2 compta2 compta 4096 déc. 15 08:42 PatriqueEr?k

---

On constate que l'identifiant lié à l'utilisateur passe bien en paramètre lors du chown

Code:

---

own -R "$Identifiant":"$Service" "/home/$Service/$Prenom$Nom"

---

Bref mon problème se trouve dans cette partie du code :

Code:

---

1 2 3 4 5 6 7 8 9

if [ $Permanent = "oui" ];then #echo "$Permanent" pwd=`curl --silent http://www.sethcardoza.com/api/rest/tools/random_password_generator` useradd "$Identifiant" -p "$pwd" -d "/home/$Service/$Prenom$Nom" -e "" -g "$Service" chown -R "$Identifiant":"$Service" "/home/$Service/$Prenom$Nom" else useradd "$Identifiant" -p "$pwd" -d "/home/$Service/$Prenom$Nom" -e "$date_expire" -g "$Service" chown -R "$Identifiant":"$Service" "/home/$Service/$Prenom$Nom" fi

---

J'ai l’impression que dans ma boucle if il n'arrive pas à check ma variable $Permanent = "oui" et passe direct au else.
Pourtant quand je fais un echo de ma variable $Permanent au dessus de ma boucle if je constate bien que celle si contient soit oui ou non .. donc je comprends pas pourquoi il me l'a saute lors de la vérification..
Je vois pas mon erreur donc si vous pouviez m'aider .. cela pourrait être sympa :calim2:

Bonjour,

Ton problème est lié à la casse: tu check si c'est "oui" alors que dans ton fichier csv c'est "Oui"

je m'en suis rendu compte quand j'ai posté en effet, mais en faite ma variable récupére soit "Oui," soit "Non,"

J'ai du coup testé en modifiant " .. = "Oui," mais rien n'y fait ..

j'ai mis un echo comme ceci :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

if [ $Permanent = "Oui," ];then echo "On est dans le THEN" pwd=`curl --silent http://www.sethcardoza.com/api/rest/tools/random_password_generator` #echo "ID:$Identifiant-PWD:$pwd-Prénom:$Prenom-Nom:$Nom" >> data-connexion.txt useradd "$Identifiant" -p "$pwd" -d "/home/$Service/$Prenom$Nom" -e "" -g "$Service" chown -R "$Identifiant":"$Service" "/home/$Service/$Prenom$Nom" else echo "On est dans le ELSE" pwd=`curl --silent http://www.sethcardoza.com/api/rest/tools/random_password_generator` useradd "$Identifiant" -p "$pwd" -d "/home/$Service/$Prenom$Nom" -e "$date_expire" -g "$Service" chown -R "$Identifiant":"$Service" "/home/$Service/$Prenom$Nom" fi

---

et comme je le pensais .. il passe direct au niveau du else, comme si il n'arrivait pas à effectuer le check ou qu'il ne fait pas le check correctement ... :(

Ok, comme le "Oui," ou "Non," sont les dernier champs, tu dois avoir un fichier au format dos, donc le "Oui," est en fait "Oui,^M"
qui est un control M ou "\r"

Tu peux le verifier via:

Code:

---

cat -A /home/toto/fichier-edit.csv

---

Si c'est le cas, essaye de modifier la ligne:

Code:

---

done < /home/toto/fichier-edit.csv

---

par

Code:

---

done < <(sed -e 's/\r//' /home/toto/fichier-edit.csv)

---

Enfin, si ton shell supporte cette écriture (bien respecter l'espace entre les 2 "<")

Ok je vais essayé ça et je te fais un retour :)

bonjour,

un test de ce type est aussi possible (dans ton cas)

Code:

---

if [[ ${Permanent,,} =~ ^oui ]]; then

---

a faire aussi sur "service" (forcer minuscule) car j'ai remarqué que tu as compta et Compta :aie:
de plus ne pas oublier d'exclure l’entête

l'appel a un service web pour générer un mot de passe est un peu fort ?
avec /dev/urandom et/ou md5sum c'est possible

+1 pour papajoker, voici un exemple avec urandom:

Code:

---

1 2 3

$ XX=$(< /dev/urandom tr -dc _A-Z-a-z-0-9%./\| | head -c${1:-8}) $ echo $XX FFNwj.tw

---

Bon après avoir réalisés plusieurs tests, j'ai finalement décidé d'utiliser en début de script :

dos2unix /home/toto/fichier-edit.csv

Et ainsi de filtrer ma variable $Permanent sur = "Oui" et cela fonctionne parfaitement maintenant... encore merci pour le rappel sur le "cat -A .." :aie:

Merci Papajoker je vais tester le test de cet façon histoire de voir comment celui-ci fonctionne et le comprendre ^^ (je commence à peine le shell :calim2:)

Par contre pour le répertoire Compta et compta .. j'avais eu la même réflexion que toi .. en me disant "tiens le prof veux nous piéger (création de doublon etc..)" mais justement quand je lui ai montré il m'a dit que en effet cela était normal et qu'il voulait bien 2 folders "compta" et "Compta". ce que je trouve totalement ... bref :aie:

Et pour la partie password je voulais faire quelques choses de vraiment "fun" pour dire "a tiens il a eu l'idée de faire ça comme sa". Je me suis dit que cela pouvait être fun (malgré que cela doit ajouter une certaines latence au script donc loin de moins l'idée de l'avoir fait dans l'optique de la performance:calim2:).


Merci pour votre aide maintenant je dois faire une sorte de fonction debug pour faire un logfile pour ce script..

Bonjour à vous je reviens vers vous pour encore un petit soucis dans ma boucle au niveau création utilisateur.

Je souhaitais ajouter un deuxième check au début de ma boucle if, si ma condition Permanent == "Oui" et que $Identifiant est déjà présent dans le fichier /etc/passwd then je créer le user.

j'ai pour le coup écris ceci mais sans succès :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

check=`cat /etc/passwd | cut -d: -f1 | grep $Identifiant` echo "$check CECI EST LE TEST ECHO"; if [ "$Permanent" == "Oui" ] || [ "$check" != "$Identifiant" ];then pwd=`curl --silent http://www.sethcardoza.com/api/rest/tools/random_password_generator` echo "$date_time ---- ID: $Identifiant -- PWD: $pwd -- Prénom: $Prenom -- Nom: $Nom -- Acc.Perma: $Permanent" >> data-connexion.txt useradd "$Identifiant" -p "$pwd" -d "/home/$Service/$Prenom$Nom" -e "" -g "$Service" chown -R "$Identifiant":"$Service" "/home/$Service/$Prenom$Nom" elif [ "$Permanent" == "Non" ] || [ "$check" != "$Identifiant" ];then pwd=`curl --silent http://www.sethcardoza.com/api/rest/tools/random_password_generator` echo "$date_time ---- ID: $Identifiant -- PWD: $pwd -- Prénom: $Prenom -- Nom: $Nom -- Acc.Perma: $Permanent" >> data-connexion.txt useradd "$Identifiant" -p "$pwd" -d "/home/$Service/$Prenom$Nom" -e "$date_expire" -g "$Service" chown -R "$Identifiant":"$Service" "/home/$Service/$Prenom$Nom" else echo "Compte EXISTANT"; fi

---

car le fait d'utiliser uniquement le test sur la variable $Permanent , me spam mon fichier data-connexion car à chaque fois que je relance le script celui-ci tente toujours de créer les utilisateurs déjà présents (vue qu'il n'y avais pas de check de présence du user) et du coup cela me fausse mon fichier data étant donnée que le curl lui aussi s’exécute à nouveau pour un même user.

Du coup je peux me retrouver avec plusieurs fois le même utilisateurs dans mon fichier data-connexion.txt avec un mdp différents pour le même compte d'où mon intérêt à ajouter un 2eme test check sur la variable $check.

Ainsi si le test $Permanent == Oui ET $check n'est pas égal à $Identifiant alors je créer l'utilisateur sinon j'affiche juste "Compte existant".

Je sais pas si je suis très clair et si il est faisable de faire un double check comme ceci.:calim2:

Si tu veux A && B, pourquoi fais-tu donc un A || B ?
|| c'est pour ou
&& c'est pour et

En tout cas, dans ton

Code:

---

if [ "$Permanent" == "Oui" ] || [ "$check" != "$Identifiant" ];then

---

Oui est suffisant pour rentré dans le bloc de commande, l'autre test n'est même pas testé, il ne le sera que si le premier est faux.

oh je suis vraiment un boulet ..... j'avais utilisé || pour faire un test hier soir tard dans la nuit ......

je me fatigue tout seul ..... quand cela saute au yeux on le vois pas (why!)

du coup j'ai remplacé || par && ,
justement comme sa je lui indique "si tu as Permanen == Oui ET $check différent de $Identifiant" alors tu exécute les commandes de création utilisateurs sinon j'affiche juste "COMPTE Existant"

car dans ma variable $check je récupère le résultat de mon cat au dessus.

En tout cas avec le bon opérateur LOGIQUE .. sa coule de source maintenant !:mouarf:

Bonjour :coucou:

Nota bene: grep utilise des expressions régulières (regular expressions = regex).
Donc quand tu filtres l'identifiant, je te conseille de mettre un signe de début de ligne '^' et un signe de fin de ligne '$'.
Sinon, "Jojo" est contenu dans "BadJojo" et ton check retourne plusieurs lignes.

Code:

---

check=$(cat /etc/passwd | cut -d: -f1 | grep "^${Identifiant}$" )

---

perso j'utiliserais plutôt une seule commande grep et ferais le test "check" dans la condition, cela ma parait plus facile a comprendre :

Code:

---

if [[ $Permanent == "Oui" && !($(grep "^${Identifiant}:" /etc/passwd)) ]]; then

---

de plus utiliser uniquement le code retour de grep (0 = trouvé) et non stdout

ps: utiliser la case dans les nom de variable :weird:

Une autre alternative:

Code:

---

if [[ "$(grep -c ^${Identifiant}: /etc/passwd)$Permanent" = "0Oui" ]]; then

---

Citation:

---

Envoyé par papajoker

perso j'utiliserais plutôt une seule commande grep et ferais le test "check" dans la condition, cela ma parait plus facile a comprendre :

Code:

---

if [[ $Permanent == "Oui" && !($(grep "^${Identifiant}:" /etc/passwd)) ]]; then

---

de plus utiliser uniquement le code retour de grep (0 = trouvé) et non stdout

ps: utiliser la case dans les nom de variable :weird:

---

Et pourquoi pas, plus simplement:

Code:

---

if [ "$Permanent" = "Oui" ] && grep -q "^${Identifiant}:" /etc/passwd; then

---