Spam: course à l'armement

Aller au contenu
Aller au menu
Aller à la recherche

Calendrier

« janvier 2006 »
lun	mar	mer	jeu	ven	sam	dim
						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

Liens

Papotages

Syndication

Par Nono | mercredi 18 janvier 2006 à 15:35 | Geekattitude | #184 | rss

Ils font chier ces spammeurs, ils continuent malgré le rel="nofollow"

Alors je n'ai pas le choix, je filtre les mots clef sur les commentaires. Désormais, tout commentaire contenant "drugstore", "pills", "phentermine", "pharmacy", "medicine", "viagra", "cialis", "xanax", "tramadol" ou "phendimetrazine" est silencieusement ignoré. Cette liste, bien sur, va s'allonger.

Pour les curieux, j'ai modifié le fichier (DotClear 1.2.1), plus exactement la fonction addComment, en ajoutant à la ligne 1935 :

$mots_interdits = array ("drugstore", "pills", "phentermine", "pharmacy", "medicine", "viagra", "cialis", "xanax", "tramadol", "phendimetrazine");
foreach ($mots_interdits as $i)
{
if (strpos ($content, $i) || strpos ($content, strtoupper($i)))
{
// On fait semblant d'accepter le commentaire, mais poubelle!
return true;
}
}

Je modifierais plus tard ce morceau de code, car ça me mail du coup quand même les commentaires SPAMs.

Commentaires

1. Le mercredi 18 janvier 2006 à 16:04, par spontex

Hello,
comme tous tes mots interdits sont en minuscules :

if (strpos (strtolower($content), $i))

serait plus simple.
De plus si ton commentaire commence par l'un de ces mots interdits, strpos retourne 0, et donc le test devrait échouer non ? :-)

2. Le mercredi 18 janvier 2006 à 16:14, par Nono

Effectivement, ta solution est plus simple! Ce que j'ai fait n'est qu'un rapide hack guidé plus par l'énervement que le réflexion :D

Le test fonctionne si le commentaire commence par l'un des mots interdits (j'ai vu tes essais ;-), et si j'ai bien compris c'est logique : PHP fait la différence ou pas entre 0 et FALSE selon le contexte (0 et false sont différents, mais ils peuvent tous les deux être interprétés comme comme étant 0, c'est pour ça qu'en PHP il ne faut jamais utiliser 0 à la place de false, notamment dans les retours de fonction).

if (strpos(...)) est équivalent à if(strpos(...) === false)

Note le triple égal, car avec un double égal, ça ne fonctionnerait pas (le test if (strpos(...) == false est foireux, lui)

3. Le mercredi 18 janvier 2006 à 16:40, par Spontex

Hello,
Je suis pas convaincu, la confusion entre 0 et false pour strpos est un piège courant. Il y a d'ailleurs un warning à ce sujet dans la doc en ligne PHP : "Cette fonction peut retourner FALSE, mais elle peut aussi retourner une valeur équivalent à FALSE, utilisable dans une condition if simple. Utilisez l' opérateur === pour tester la valeur exacte de cette fonction."
Dans les commentaires on en voit même qui font des :
if (strpos(' '.$string,$substring) > 1) pour être sûrs de leur coup :-))

4. Le mercredi 18 janvier 2006 à 17:03, par Nono

Si si! La preuve, ça fonctionne ;-)

En PHP, il n'y a pas de typage fort, ce qui authorise dans notre cas la fonction strpos a renvoyer un entier s'il a trouvé l'index (0, 1, 2...) ou un booléen valant faux s'il n'a rien trouvé (cf la doc: Si needle n'est pas trouvée, la fonction retourne FALSE). Regarde l'exemple 1 de la doc, c'est clair!

Les commentaires que tu cites dans la doc sur cette fonctions ont simplement été écrits par des gens qui n'arrivent pas à conceptualiser le typage faible, ou qui n'arrivent pas à envisager une manipulation de chaine de caractère différemment qu'en C (en C, false et zéro, c'est pareil, car le C ne connait pas le type booléen à la base, du moins selon la norme ISO99).

5. Le mercredi 18 janvier 2006 à 17:08, par Nono

Je m'auto-confirme, je viens de matter les sources de php 4.4.2 8-)

6. Le mercredi 18 janvier 2006 à 21:05, par Spontex

Hello,
Une valeur de retour FALSE signifie que la chaîne n'a pas été trouvée.
Une valeur de retour 0 signifie que la chaîne a été trouvée, au début du commentaire.

Ces deux situations se retrouvent dans la même branche d'un "if" qui servait pourtant à les différencier.

Un exemple :

<?php

function teste_commentaire($content) {
$mots_interdits = array ("mot1", "mot2");
foreach ($mots_interdits as $i) {
if (strpos ($content, $i) || strpos ($content, strtoupper($i)))
echo "Spam détecté";
}
}

teste_commentaire("Jacques a dit ... mot1 is good");
teste_commentaire("mot1 is good");

?>

7. Le mercredi 18 janvier 2006 à 21:10, par Spontex

PS : signé Spontex, encore plus chiant que les spammeurs :-))

8. Le dimanche 9 avril 2006 à 01:04, par ambien side effect

buy ambien online

You can ambien cr here buy ambien online.

Ajouter un commentaire

Les commentaires pour ce billet sont fermés.

( Le petit robot de Toulouse )

Calendrier

Catégories

Archives

Liens

Papotages

Syndication