regression require htmlawed, lien sans http dans Security, bouton remonter

3 files changed, 27 insertions, 33 deletions

diff --git a/controller/Security.php b/controller/Security.php
index 15c44b2..ac1149e 100644
--- a/controller/Security.php
+++ b/controller/Security.php
@@ -1,6 +1,9 @@
 <?php
 // controller/Security.php
 
+// sécurité faille XSS avec htmLawed
+require('lib/htmlawed/htmLawed.php');
+
 class Security
 {
         private static $configHtmLawed = array(
@@ -13,17 +16,38 @@ class Security
 
         public static function secureString(string $chaine): string
         {
-                // sécurité faille XSS avec htmLawed
-                require('lib/htmlawed/htmLawed.php');
-
                 $chaine = htmLawed($chaine, self::$configHtmLawed, self::$specHtmLawed);
             $chaine = trim($chaine); // supprimer espaces, tabulations et sauts de ligne en début et fin de chaine (pour l'entrée de l'éditeur)
             return $chaine;
         }
 }
 
+// erreurs à la création des mots de passe
 function removeSpacesTabsCRLF(string $chaine): string
 {
         $cibles = [' ', "\t", "\n", "\r"]; // doubles quotes !!
         return(str_replace($cibles, '', $chaine));
 }
+
+// lien sans http://
+function fixLinks($data)
+{
+    // si une adresse est de type "domaine.fr" sans le http:// devant, le comportement des navigateurs est de rechercher un fichier comme si mon adresse commençait par file://
+    // tomber ainsi sur une page d'erreur est parfaitement déroutant
+    // une erreur 404 est détectée et le visiteur redirigé, mais ça ne règle pas le problème
+
+    // regex pour détecter les balises <a> et ajouter http:// au début des liens si nécessaire
+    $pattern = '#(<a.*href=")((?!https?://).*)(".*>)#';
+    $remplacement = '$1http://$2$3';
+    // le "while" est ici parce que preg_replace s'arrête après avec trouvé et modifié un pattern, pourquoi??
+    while(preg_match($pattern, $data))
+    {
+        $data = preg_replace($pattern, $remplacement, $data);
+    }
+
+    // 2ème problème, en train d'écrire un article, l'utilisateur clique sur ce lien qu'il vient de créer et qui apparaît dans l'infobulle, un nouvelle onglet apparaît
+    // solution: fermer ce nouvel onglet avec echo '<SCRIPT>javascript:window.close()</SCRIPT>';
+    // comment déclencher le JS? en faisant qu'une erreur 404 causée pour cette raison soit particulière?
+
+    return($data);
+}
\ No newline at end of file
diff --git a/controller/ckeditor.php b/controller/ckeditor.php
index d6ea8ad..43fd2e2 100644
--- a/controller/ckeditor.php
+++ b/controller/ckeditor.php
@@ -142,32 +142,3 @@ function submitCKeditor()
     header('Location: index.php?page=' . $_GET['page']);

     exit();

 }

-

-

-// lien sans http://

-// si une adresse est de type "domaine.fr" sans le http:// devant, le comportement des navigateurs est de rechercher un fichier comme si mon adresse commençait par file://

-// tomber ainsi sur une page d'erreur est parfaitement déroutant

-// une erreur 404 est détectée et le visiteur redirigé, mais ça ne règle pas le problème

-// 2ème problème, en train d'écrire un article, l'utilisateur clique sur ce lien qu'il vient de créer et qui apparaît dans l'infobulle, un nouvelle onglet apparaît

-// solution: fermer ce nouvel onglet avec echo '<SCRIPT>javascript:window.close()</SCRIPT>';

-// le .htaccess doit pour ça détecter que l'erreur 404 est particulière

-

-function fixLinks($data)

-{

-    // regex pour détecter les balises <a>, analyser les liens à l'interieur et les modifier si nécessaire

-    $pattern = '#<a href=".*">.*</a>#';

-    

-    if(preg_match($pattern, $data))

-    {}

-

-    //$remplacement = 'http://$0';

-    //$data = preg_replace($pattern, $remplacement, $data);

-    

-    // contrairement à ce qui dit la doc

-    // https://www.php.net/manual/fr/filter.filters.validate.php

-    // une adresse du type "domaine.fr" ne passe pas le filtre

-    // il renvoie false, ce qui nous arrange ici!

-    //var_dump(filter_var($link, FILTER_VALIDATE_URL));

-

-    return($data);

-}

diff --git a/controller/media.php b/controller/media.php
index c6682df..27a8bb3 100644
--- a/controller/media.php
+++ b/controller/media.php
@@ -48,7 +48,6 @@ function mediaSubmit($chaine)
     // c'est du complètement calqué sur le javascript, avec une syntaxe PHP

     // et en plus elle est super vieille, elle ne connaît pas les balises HTML5 !

     // mais une balise c'est une balise donc ça marche quand même

-    // la doc de PHP est comme toujours plutôt bonne, elle indique les types de données manipulées

     // https://www.php.net/manual/fr/book.dom.php

     $dom = new DOMDocument;