Discussion :

[psychoBob]

Bonjour,

Qui peut me dire pourquoi cette requête plante avec obstination ?

Code :

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

1
2
3
4
5
6
 $connexionRatee=
"UPDATE ipBloque 
SET tentative=tentative+1, dateLimite='".$dateUnix."'  
WHERE ip='".$ip."' AND dateLimite > '".$dateMaxi."'
";
mysql_query($connexionRatee);

ça ne donne pas de message d'erreur, mais ça n'update pas non plus.

[Julien.alkaza]

SET tentative=tentative+1

Je pense que ca vient de là...
Je ne crois pas que tu puisse utiliser la valeur d'un champs comme ca...
Il faut que tu récupère sa valeur via un "SELECT tentative FROM IPBloquee WHERE IP = '$IP';";

Puis après tu incrémente ta valeur tentative ( $tentative++, et seulement ensuite tu fais un UPDATE avec tentative = '$tentative'...

[billoum]

Salut,

Pê qu'aucun enregistrement ne satisfait la condition.
Fais un select avec la même clause WHERE pour voir si ca te retourne des enregistrements.

[psychoBob]

En fait c'est tout le script qui foire...

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
31
32
33
34
35
36
37
//on récupère l'IP
$ip = $_SERVER["REMOTE_ADDR"];
// la date unix
$dateUnix=time();
// la date unix - 15 minutes
$dateMaxi=$dateUnix-900;
//on va chercher si des ip ont été enregistrée dans les 15 minutes.
$affiche="
SELECT ip  FROM ipBloque 
WHERE ip='".$ip."' AND dateLimite > '".$dateMaxi."'";
$resultat = mysql_query("$affiche");
$ipBloque=mysql_num_rows($resultat);
//si on a trouvé aucune ip, on insère celle ci (ça n'insère rien).
if($ipBloque=0)
{
$connexionRatee="INSERT INTO `ipBloque` (`ip`,`dateLimite`,`tentative`) 
VALUES
(
'$ip','$dateUnix','1'
)";
mysql_query($connexionRatee);
}
//si on a trouvé un résultat, on update le tuple en 
incrémentant de 1 le nombre de tentative, 
ainsi que la date, pour 
bloquer 15 minute si tentative est à 5 (ça c'est dans un autre script): 
ça n'update rien non plus.
if($ipBloque==1)
{
$connexionRatee="
UPDATE ipBloque 
SET tentative='tentative+1', dateLimite='".$dateUnix."'  
WHERE ip='".$ip."' AND dateLimite > '".$dateMaxi."'";
mysql_query($connexionRatee);
;}
mysql_free_result($resultat);

[psychoBob]

Salut,

Pê qu'aucun enregistrement ne satisfait la condition.
Fais un select avec la même clause WHERE pour voir si ca te retourne des enregistrements.

Oui mais comme je viens de dire dans la réponse que j'écrivais pendant que tu faisais du même, si ça ne trouve rien, ça doit quand même insérer un tuple, justement. Et si ça en trouve un, celui qui vient d'être inséré, alors on l'update.

Je me demande si ça ne viendrait pas du fait que les variables tout en haut ($dateMaxi et $ip) ne sont pas transmise à l'intérieur du script, mais je ne comprend pas pourquoi alors.

[Julien.alkaza]

Tes requêtes marchent ou pas??
J'entends par là dans PHPMyAdmin (ou autre)??

Sinon, tu n'as pas besoin de faire tout un fouilli comme celui là :

ip='".$ip."'

Si tu mets ta requête entre "", tu fais juste

Code :

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

"...WHERE ip = '$ip'..."

et c'est bon!!

[psychoBob]

La requête insert fonctionne bien dans phpMyAdmin, avec des chiffres comme 1 à la place des variables.
Mais dans le script ça ne fonctionne pas.

L'update j'ai pas testé dans phpMyAdmin, je veux déjà régler le INSERT.

[billoum]

T'as fait :

Code :

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

mysqlquery($requete) or die($requete.':<br>'.mysql_error());

après chaque requête?
Et aussi (même si jpense pas que ca changera gd chose), c'est une faute de frappe là (manque '=='):

Code :

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

if($ipBloque=0)

?

[psychoBob]

Bon alors, c'est réglé pour ce script, il manquait un = à if(ipBloque=0).

Maintenant c'est celle là qui foire :

Code :

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

1
2
3
4
5
6
 
"
SELECT ip  
FROM ipBloque 
WHERE ip='".$ip."' AND dateLimite > '".$dateMaxi."' AND tentative <= 5
"

ça doit être au niveau du 5 je pense...



**edit**

On a posté ensemble, Billoum

Et aussi (même si jpense pas que ca changera gd chose), c'est une faute de frappe là (manque '=='):

Code :

if($ipBloque=0)

Bah si c'était radical même comme erreur

[Julien.alkaza]

Tiens...j'l'avais pas vu celui là

[billoum]

Ben en fait j'avais testé chez moi un if pareil mais ca n'empêchait pas de rentrer dans la boucle ( ca t'aurait juste empêché de rentrer dans la seconde vu que 0 serait affecté à la variable, enfin bon...).

Sinon, pour tester tes requêtes,est-ce que tu fais:

Code :

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

mysqlquery($requete) or die($requete.':<br>'.mysql_error());

?

[psychoBob]

Je viens de faire le mysql_error mais ça ne retourne pas d'erreur. ça ne trouve pas le tuple, pourtant il y en a un qui correspond.

ça doit encore être une erreur bidon, mais bon...

[psychoBob]

Bon déjà l'opérateur était à l'envers, c'était >= et non <= .


Mais c'est pas ça quand même

[Julien.alkaza]

Essaye ce que je te dis...
remplace tes '".$ip."' par '$ip'

Ca simplifie grandement les choses....et ca vient peut être de là...

EDIT : par "datelimite" tu entends quoi dans ta table??

[psychoBob]

J'ai réussi, j'avais pas changé la logique du script, je fais toujours des if $ipBloque<=5 alors qu'il fallait désormais juste tester si il y a avait un tuple donc if$ipBloque==1.

Pour les '".$ip."' à remplacer par '$ip', on m'avait expliqué que les ".." renforçait la sécurité en gênant les injection sql.


Pour le dateLimite, t'étais sur la piste Julien.alkaza, mais sans le script en entier...

Merci à tous en tout cas.