Changeset 65567 in spip-zone

Timestamp:

Sep 5, 2012, 1:22:57 PM (8 years ago)

Author:

cedric@…

Message:

Refactoring et double detection des liens deja dans des spams : si on refuse pas le message a la saisie (seuil <3), on le mets quand meme en spam des qu'un lien est deja dans un spam

Location:

_plugins_/nospam

Files:

• inc/nospam.php (modified) (1 diff)
• nospam/verifier_formulaire_forum.php (modified) (1 diff)
• nospam_pipelines.php (modified) (2 diffs)
• plugin.xml (modified) (1 diff)

_plugins_/nospam/inc/nospam.php

@@ -173 +173 @@
 }
 
+/**
+ * Compare les domaines des liens fournis avec la presence dans la base
+ *
+ * @param array $liens
+ *   liste des liens html
+ * @param int $seuil
+ *   seuil de detection de presence : nombre d'enregistrement qui ont deja un lien avec le meme domaine
+ * @param string $table
+ *   table sql
+ * @param array $champs
+ *   champs a prendre en compte dans la detection
+ * @param null|string $condstatut
+ *   condition sur le statut='spam' pour ne regarder que les enregistrement en statut spam
+ * @return bool
+ */
+function rechercher_presence_liens_spammes($liens,$seuil,$table,$champs,$condstatut=null){
+        include_spip("inc/filtres");
+
+        if (is_null($condstatut))
+                $condstatut = "statut=".sql_quote('spam');
+        if ($condstatut)
+                $condstatut = "$condstatut AND ";
+
+        $hosts = array();
+        foreach ($liens as $lien){
+                $url = extraire_attribut($lien,"href");
+                if ($parse = parse_url($url)
+                  AND $parse['host'])
+                        $hosts[] = $parse['host'];
+        }
+
+        $hosts = array_unique($hosts);
+        $hosts = array_filter($hosts);
+
+        // pour chaque host figurant dans un lien, regarder si on a pas deja eu des spams avec ce meme host
+        // auquel cas on refuse poliment le message
+        foreach($hosts as $h){
+                $like = " LIKE ".sql_quote("%$h%");
+                $where = $condstatut . "(".implode("$like OR ",$champs)."$like)";
+                if (sql_countsel($table,$where)>=$seuil){
+                        return $h;
+                }
+        }
+        return false;
+}
 ?>

_plugins_/nospam/nospam/verifier_formulaire_forum.php

@@ -45 +45 @@
                   AND count($infos['liens'])){
 
-                        $hosts = array();
-                        foreach ($infos['liens'] as $lien){
-                                $url = extraire_attribut($lien,"href");
-                                if ($parse = parse_url($url)
-                                  AND $parse['host'])
-                                        $hosts[] = $parse['host'];
-                        }
 
-                        $hosts = array_unique($hosts);
-                        $hosts = array_filter($hosts);
-
-                        // pour chaque host figurant dans un lien, regarder si on a pas deja eu des spams avec ce meme host
-                        // auquel cas on refuse poliment le message
-                        foreach($hosts as $h){
-                                if (sql_countsel("spip_forum","statut=".sql_quote('spam')." AND texte LIKE ".sql_quote("%$h%"))>=_SPAM_URL_MAX_OCCURENCES){
-                                        spip_log("Refus message de forum qui contient un lien vers $h","nospam");
-                                        $flux['data']['texte'] = _T('nospam:erreur_url_deja_spammee');
-                                        break; // pas la peine de continuer avec les autres liens
-                                }
+                        if ($h = rechercher_presence_liens_spammes($infos['liens'],_SPAM_URL_MAX_OCCURENCES,'spip_forum',array('texte'))){
+                                spip_log("Refus message de forum qui contient un lien vers $h","nospam");
+                                $flux['data']['texte'] = _T('nospam:erreur_url_deja_spammee');
                         }
                 }

_plugins_/nospam/nospam_pipelines.php

@@ -155 +155 @@
                                         // s'il y a du contenu caché avec des styles => spam direct
                                         $flux['data']['statut'] = 'spam';
-                                }elseif ($infos['nombre_liens'] > 0) {
+                                }
+                                elseif ($infos['nombre_liens'] > 0) {
                                         // si un lien a un titre de moins de 3 caracteres, c'est louche...
                                         if ($infos['caracteres_texte_lien_min'] < 3) {
@@ -170 +171 @@
                                                         $flux['data']['statut'] = $stat;
                                                         spip_log("\t".$flux['data']['auteur']."\t".$GLOBALS['ip']."\t"."requalifié en ".$stat." car nombre_liens >= ". $s,'nospam');
-                                                }       
+                                                }
+
+                                        if ($flux['data']['statut'] != 'spam'){
+                                                $champs = array_unique(array('texte',$champ));
+                                                if ($h = rechercher_presence_liens_spammes($infos['liens'],1,'spip_forum',$champs)){
+                                                        $flux['data']['statut'] = 'spam';
+                                                        spip_log("\t".$flux['data']['auteur']."\t".$GLOBALS['ip']."\t"."requalifié en spam car lien $h deja dans un spam",'nospam');
+                                                }
+                                        }
                                 }
                         }
+
 
                         // verifier qu'un message identique n'a pas ete publie il y a peu

_plugins_/nospam/plugin.xml

@@ -10 +10 @@
         <icon>images/nospam-32.png</icon>
         <licence>(c) 2008 GPL</licence>
-        <version>1.0.1</version>
+        <version>1.0.2</version>
         <etat>stable</etat>
         <description><multi>