Discussion :

[Acidix]

Bonjour à tous, Je souhaite récupérer les adresses Ips qui se sont connectés à mon serveur Apache et les stockées dans ma base de données SQLserver (J'utilise ODBC)

Code :

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

1
2
ipConnect=$(cat /var/log/apache2/access.log | cut -f1 -d' ' | sort | uniq) #la commande me donnant les ips (sans doublons) stockée dans la variable ipConnect
echo -e "INSERT INTO dbo.Logs (ipLog) VALUES ('$ipConnect');" | isql DATABASE MDP LOG -v

la connexion se réalise, mais insère des champs vides dans la base de données puisque la variable n'est pas prise en compte en SQL, je voulais connaitre votre avis pour savoir si c'est bien un problème de lecture de la variable et comment résoudre le problème.
Merci pour la réponse !!!!

[Flodelarab]

Bonjour,

Pour le mot clé VALUES, le séparateur n'est pas l'espace (mais la virgule ou le point-virgule, je ne sais plus). Pourtant, dans ta variable, aucune virgule ou point-virgule.

Ensuite, il faudrait vérifier que isql accepte qu'on lui mette un fichier en entrée. Rien n'est moins sûr. Il sera peut-être nécessaire d'utiliser While do done < <() ou alors xargs -ITT commande TT.

[Acidix]

Merci pour cette réponse, du coup j'ai pris en compte vos remarques et je me dis qu'il faudrait mieux que je stocke chaque Ips dans des variables différentes

Code :

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

1
2
3
4
5
6
7
8
9
10
ipConnect=$(cat /var/log/apache2/access.log | cut -f1 -d' ' | sort | uniq)#toutes les adresses qui se sont connectées au serveur
nbrConnect=$(echo "$ipConnect" | wc -l)#nombres de connections au serveur
i=1
while [ "$i" -le "$nbrConnect" ]#tant que i<aux nombres de connections (soit aux nombres de lignes de la variable ipConnect)
do
ip"$i"=$(echo "$ipConnect" | grep -E ^"$i")#la variable ipN stockera la N ème lignes d'adresses <== cette commande ne marche pas 
echo "ip$i"#on affiche le resultat afin de vérifier si la commande fonctionne
echo -e "INSERT INTO dbo.Logs (ipLog) VALUES ('$ip"$i"');" | isql DATABASE MDP LOG -v 
i="$i"+1
done

la je sépare les adresses, comme ça j'aurais un champ=une adresse dans ma BDD, mais les variables Bash ne sont pas prisent en compte dans le SQL ...

[N_BaH]

il faudrait mieux que je stocke chaque Ips dans des variables différentes

très mauvaise idée

un tableau pourrait aider :

Code :

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

1
2
3
4
sep=#définir virgule ou point-virgule
ar=( $(cut -f1 -d' ' /var/log/apache2/access.log | sort -u) )
oIFS="$IFS"; IFS="$sep"; printf -v values "${ar[*]}"; IFS="$oIFS"
echo "$values"

[Acidix]

ah bah oui c'est propre comme ça ...
Du coup une fois que mes Ips seront dans le tableau, je pourrais directement les insérer dans ma BDD avec :

Code :

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

INSERT INTO dbo.Logs (ipLog) VALUES ('$values');

Et la variable values sera bien pris en compte par sql ? enfin ça ne sera pas des champs vides que SQLserver va me créer ?
Bon bah je testerais ça demain ! merci beaucoup!

[Flodelarab]

Personnellement, je prendrais la sortie de la liste des IPs et j'en ferais un texte bien formaté pour le passer à la commande isql. Mais j'ai l'impression de me répéter.

Utiliser des quotes plutôt que des double quotes me semble une mauvaise idée.

[N_BaH]

avec des guillemets (double-quotes) autour, pas de problème :

Code :

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

1
2
3
4
5
$ var="la la"
$ echo "tra '$var' lère"
tra 'la la' lère
$ echo tra '$var' lère
tra $var lère

par contre l'option -e n'est peut-être pas souhaitable (elle active l'interprétation des caractères d'échappement). Ce comportement ne pourrait-ils poser des problèmes ?

[Acidix]

Mais enfaite c'est pas possible d'insérer toutes mes données sans faire de boucle .. si je rentre une ip ça fonctionne mais la il y a plusieurs valeurs :

Code :

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

echo -e "INSERT INTO dbo.Logs1 (ipLog) VALUES ('$values');" | isql DATABASE LOG MDP -v

je vais inséré toutes mes ips avec une virgule puisque sep=,
on aura genre "INSERT INTO dbo.Logs1 (ipLog) VALUES ('192.168.10.105,192.168.10.107,192.10.135');" mais la ma base de données va comprendre que je souhaite inséré ces données dans un seul champs non ? mais après mon problème de départ est résolut puisque un tableau bash est bien en compte dans ma requête sql ! Merci pour ces réponses

[Acidix]

c'est bon mon problème est résolut !! ça marche impec, j'ai simplement fait ma requête SQL dans une boucle while! MERCI

[Flodelarab]

La boucle est inutile puisque VALUES peut prendre plusieurs enregistrement d'un coup.

Mais comme tu as résolu le problème, c'est bon.