fichiers index.php

4 files changed, 92 insertions, 62 deletions

diff --git a/controller/Security.php b/controller/Security.php
index c53cdc6..15c44b2 100644
--- a/controller/Security.php
+++ b/controller/Security.php
@@ -1,9 +1,6 @@
 <?php
 // controller/Security.php
 
-// sécurité faille XSS avec htmLawed
-require('lib/htmlawed/htmLawed.php');
-
 class Security
 {
         private static $configHtmLawed = array(
@@ -16,6 +13,9 @@ 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;
diff --git a/controller/backup.php b/controller/backup.php
index 1477c55..2cfccaa 100644
--- a/controller/backup.php
+++ b/controller/backup.php
@@ -103,55 +103,63 @@ function restaurer($from)
 {

         $title = 'Restauration des données';

         $message = '';

+        global $archiveFormat;

 

         // recharger la même page en écrivant les données

         if(isset($_FILES['archive']) && $_FILES['archive']['error'] == 0)

         {

                 // détecter le format (zip ou autre)

+                if($archiveFormat == 'zip')

+                {

+                        // une copie du zip est conservée dans data/ au cas où

+                        move_uploaded_file($_FILES['archive']['tmp_name'], 'data/' . $_FILES['archive']['name']);

+                        chmod('data/' . $_FILES['archive']['name'], 0666);

 

-                // une copie du zip est conservée dans data/ au cas où

-                move_uploaded_file($_FILES['archive']['tmp_name'], 'data/' . $_FILES['archive']['name']);

-                chmod('data/' . $_FILES['archive']['name'], 0666);

-

-                $nomFichier = 'data/' . $_FILES['archive']['name'];

+                        $nomFichier = 'data/' . $_FILES['archive']['name'];

 

-                // extraction

-                try

-                {

-                        $Zip = new ZipArchive();

-                }

-                catch (Throwable $e) // l'extension zip n'est pas activée

-                {

-                        echo($e);

-                        die();

-                }

-                if($Zip->open($nomFichier, ZipArchive::RDONLY) === TRUE)

-                {

-                        $j = 0;

-                        for($i = 0; $i < $Zip->numFiles; $i++)

+                        // extraction

+                        try

+                        {

+                                $Zip = new ZipArchive();

+                        }

+                        catch (Throwable $e) // l'extension zip n'est pas activée

                         {

-                                $nomEntree = $Zip->getNameIndex($i);

-                                if($Zip->extractTo('.', $nomEntree) === TRUE)

+                                echo($e);

+                                die();

+                        }

+                        if($Zip->open($nomFichier, ZipArchive::RDONLY) === TRUE)

+                        {

+                                $j = 0;

+                                for($i = 0; $i < $Zip->numFiles; $i++)

                                 {

-                                        $j++;

+                                        $nomEntree = $Zip->getNameIndex($i);

+                                        if($Zip->extractTo('.', $nomEntree) === TRUE)

+                                        {

+                                                $j++;

+                                        }

+                                        else

+                                        {

+                                                $message = '<p style="color: red;" >Erreur: extraction du zip impossible.</p>';

+                                        }

+                                        chmod($nomEntree, 0666);

                                 }

-                                else

+

+                                if($j == $Zip->numFiles)

                                 {

-                                        $message = '<p style="color: red;" >Erreur: extraction du zip impossible.</p>';

+                                        $message = '<p style="color: red;" >Restauration réussie !!</p>';

                                 }

-                                chmod($nomEntree, 0666);

+                                $Zip->close();

                         }

-

-                        if($j == $Zip->numFiles)

+                        else

                         {

-                                $message = '<p style="color: red;" >Restauration réussie !!</p>';

+                                // mauvais fichier

+                                $message = '<p style="color: red;" >Erreur: Impossible d\'ouvrir l\'archive Zip."</p>';

                         }

-                        $Zip->close();

                 }

                 else

                 {

-                        // mauvais fichier

-                        $message = '<p style="color: red;" >Erreur: Impossible d\'ouvrir l\'archive Zip."</p>';

+                        // pas de module zip

+                        $message = '<p style="color: red" >Erreur: Veuillez activer l\'extension zip dans le php.ini pour pouvoir gérer les sauvegardes.</p>';

                 }

         }

         elseif(isset($_FILES['archive']) && $_FILES['archive']['error'] != 0)

diff --git a/controller/ckeditor.php b/controller/ckeditor.php
index 4b4297a..d6ea8ad 100644
--- a/controller/ckeditor.php
+++ b/controller/ckeditor.php
@@ -47,6 +47,9 @@ function submitCKeditor()
     if(isset($_POST['contenu'])) // optionnel pour discographie

     {

         $contenu = Security::secureString($_POST['contenu']);

+

+        // liens sans http:// devant

+        $contenu = fixLinks($contenu);

         

         // récupérer les liens multimedia

         //require("media.php");

@@ -142,14 +145,29 @@ function submitCKeditor()
 

 

 // lien sans http://

-// un clic sur un lien dans l'éditeur affiche une infobulle montrant l'adresse cible du lien si celle-ci a déjà été précisée

-// il est possible de cliquer sur ce lien, ce qui ouvre un onglet avec le site demandé

-// toutefois si cette adresse est de type "domaine.fr" (sans http:// devant), le navigateur ne va pas rechercher un site mais un fichier comme si mon adresse était de type file:///fichier

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

-// "ai-je perdu le texte que j'étais en train de taper?""

-// solution 1 (mauvaise): activer la redirection en cas d'erreur 404 dans le .htaccess

-// solution 2 (façon pop-up): fermer ce nouvel onglet avec echo '<SCRIPT>javascript:window.close()</SCRIPT>';

-// pour faire passer par le .htaccess l'info comme quoi la page précédente comportait un éditeur ouvert...

-

-function cleanHTML($contenu)

-{}

+// 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/installation.php b/controller/installation.php
index e8b1af8..0bd56c2 100644
--- a/controller/installation.php
+++ b/controller/installation.php
@@ -68,10 +68,19 @@ function installation()
         require('view/backup.php');

         exit();

     }

-    if(!file_exists('data/index.html'))

+    

+    function createIndexPHP($path, $droitsFichiers)

+    {

+        $content = "<?php\nheader('Location: ../index.php');\nexit();";

+        $file = fopen($path, 'w');

+        fputs($file, $content);

+        fclose($file);

+        chmod($path, $droitsFichiers);

+    }

+

+    if(!file_exists('data/index.php'))

     {

-        touch('data/index.html');

-        chmod('data/index.html', $droitsFichiers);

+        createIndexPHP('data/index.php', $droitsFichiers);

     }

     

     $listePages = array('melaine', 'discographie', 'concerts', 'presse', 'ateliers', 'liens', 'peinture', 'archives');

@@ -82,30 +91,27 @@ function installation()
             mkdir('data/' . $page);

             chmod('data/' . $page, $droitsDossiers);

         }

-        if(!file_exists('data/' . $page . 'index.html'))

+        if(!file_exists('data/' . $page . '/index.php'))

         {

-            touch('data/' . $page . '/index.html');

-            chmod('data/' . $page . '/index.html', $droitsFichiers);

+            createIndexPHP('data/' . $page . '/index.php', $droitsFichiers);

         }

         if(!file_exists('data/' . $page . '/html'))

         {

             mkdir('data/' . $page . '/html');

             chmod('data/' . $page . '/html', $droitsDossiers);

         }

-        if(!file_exists('data/' . $page . '/html/index.html'))

+        if(!file_exists('data/' . $page . '/html/index.php'))

         {

-            touch('data/' . $page . '/html/index.html');

-            chmod('data/' . $page . '/html/index.html', $droitsFichiers);

+            createIndexPHP('data/' . $page . '/html/index.php', $droitsFichiers);

         }

         if(!file_exists('data/' . $page . '/images'))

         {

             mkdir('data/' . $page . '/images');

             chmod('data/' . $page . '/images', $droitsDossiers);

         }

-        if(!file_exists('data/' . $page . '/images/index.html'))

+        if(!file_exists('data/' . $page . '/images/index.php'))

         {

-            touch('data/' . $page . '/images/index.html');

-            chmod('data/' . $page . '/images/index.html', $droitsFichiers);

+            createIndexPHP('data/' . $page . '/images/index.php', $droitsFichiers);

         }

         // if(!file_exists('data/' . $page . '/multimedia'))

         // {

@@ -119,10 +125,9 @@ function installation()
         mkdir('data/discographie/json');

         chmod('data/discographie/json', $droitsDossiers);

     }

-    if(!file_exists('data/discographie/json/index.html'))

+    if(!file_exists('data/discographie/json/index.php'))

     {

-        touch('data/discographie/json/index.html');

-        chmod('data/discographie/json/index.html', $droitsFichiers);

+        createIndexPHP('data/discographie/json/index.php', $droitsFichiers);

     }

     if(!file_exists('data/discographie/images-mini'))

     {

@@ -131,8 +136,7 @@ function installation()
     }

     if(!file_exists('data/discographie/images-mini/index.html'))

     {

-        touch('data/discographie/images-mini/index.html');

-        chmod('data/discographie/images-mini/index.html', $droitsFichiers);

+        createIndexPHP('data/discographie/images-mini/index.php', $droitsFichiers);

     }

     // le modèle donnera les droits 0666 (octal) aux nouveaux fichiers à l'intérieur des dossiers