BGC-26 amelioration protection des données SCORM

Entity/ZipFile.php

@@ -330,9 +330,9 @@ class ZipFile extends AbstractEntity
      * @param string $zipPath
      * @param string $uploadPath
      *
-     * @return array
+     * @return bool
      */
-    public function uploadZipFromPath(string $zipPath, string $uploadPath)
+    public function uploadZipFromPath(string $zipPath, string $uploadPath) : bool
     {
         if (file_exists($zipPath)) {
             // Récupération de la signature du zip

LogiproSCORMBundle.php

@@ -16,11 +16,17 @@ class LogiproSCORMBundle extends Bundle
     {
         parent::build($container);
     }
-
-    private function initialiseFolders()
+/**
+ * *creation du repertoire qui contiendra les ressources accesibles :
+ * ** stockage des paquets scorms (logipro_scorm.content_folder)
+ * * creation du fichier .htaccess qui restreindra l'acces aux ressources
+ *
+ * @return void
+ */
+    private function initialiseFolders() : void
     {
         $contentFolder = $this->container->getParameter('logipro_scorm.content_folder');
-        // creation du repertoir de reception  du contenu
+        // creation du repertoire de reception  du contenu
         if (!file_exists($contentFolder)) {
             mkdir($contentFolder, 0755, true);
         }
@@ -29,39 +35,39 @@ class LogiproSCORMBundle extends Bundle
         
         $htaccess = $contentFolder.'/.htaccess';
         // on prendra garde de doubler le \ en \\
+        $now = $today = date("Y-m-d H:i:s");
         $content=<<<EOT
+# automatically generated by LogiproSCORMBundle.initialiseFolders at $now
 Options +FollowSymlinks
 RewriteEngine On
 
 RewriteBase "$contentUrlPrefix/"
 
-# l'acces au ressource est donnée a une url qui contient un code correspondant à
-# une operation de transofrmation de la valeur du cookie
+#en cas d'acces à un fichier/existant existant laisse l'acces... 
+#...a condition d'un environnement phpsession indice (insuffisant) que c'est un utilisateur connu
+# (a ameliorer)
+RewriteCond %{REQUEST_FILENAME} -f [OR]
+RewriteCond %{REQUEST_FILENAME} -d
+RewriteCond %{HTTP_COOKIE} !PHPSESSID= [NC]
+RewriteRule .* - [L]
 
+# l'acces au ressource est donnée a une url qui contient un code correspondant à
+# une operation de transformation de la valeur du cookie
 
-#si le coockie n'existe pas alors pas droit d'acces à la ressource403 
+#si le coockie n'existe pas alors pas droit d'acces à la ressource : 403 
 RewriteCond %{HTTP_COOKIE} !SCORMBundle= [NC]
 RewriteRule .* - [R=403,L]
 
-
 #on stocke la valeur du cookie...
 RewriteCond %{HTTP_COOKIE} SCORMBundle=([^;]+)
 RewriteRule ^(.*) $1 [E=md5scormkey:%1]
 
-#RewriteRule .* https://www.logipro.com/%{ENV:md5scormkey} [L]
-
-# on recupere la clef contenu dans la chaine
+# on recupere la clef contenue dans la chaine
 RewriteCond %{REQUEST_URI} ^$contentUrlPrefix/([^/]+)/(.+)$
 RewriteRule .* - [E=key:%1,E=resource:%2]
+RewriteCond expr "md5('%{ENV:key}') == %{ENV:md5scormkey}"
 
-RewriteCond expr "md5('%{ENV:key}') = '%{ENV:md5scormkey}'"
-
-#on compare la valeur du cookie à ce qu'il y dans l'url : oui il faut connaitre un peu les expression perl
-# une reference arriere est noté \\n dans une chaine
-#RewriteCond %{ENV:md5scormkey}:%{ENV:key} ^(.*):\\1
-#RewriteCond %{REQUEST_FILENAME} !-f
 RewriteRule .* $contentUrlPrefix/%{ENV:resource} [L]
-
 EOT;
 
         $handle = fopen($htaccess, 'w');
@@ -73,6 +79,5 @@ EOT;
         if (!file_exists($packageFolder)) {
             mkdir($packageFolder, 0777, true);
         }
-
     }
 }

README.md

@@ -41,8 +41,8 @@ php bin/console doctrine:schema:update --force
 Security
 --------
 
-Package resources access has cookie authentification. It allows application access only to user 
-owning resources. For instance none would be able to access diurecly by url.
+Package resources access has cookie authentification. It allows application access only to user
+owning resources. For instance none would be able to access directly by url.
 (example : http://yourdomaine/content/SeqPostSCORM20044th/imsmanifest.xml will return a 403)
 
 You must configure the following things :
@@ -51,11 +51,11 @@ You must configure the following things :
 logipro_scorm:
   package_folder: '%kernel.project_dir%/private/packages'
   content_folder: '%kernel.project_dir%/public/content'
-  content_url_prefix: '/lms/content'
+  content_url_prefix: '/content'
 ```
 
-A special .htacces will be autogenerated in content_folder and url_prefix will be used to
-create url access to ressources, like this : https://yourdomain/content_url_prefix/CODE/ressourceofapackage
+A special .htacces file will be autogenerated in content_folder and url_prefix will be used to
+create url access to resources, like this : https://yourdomain/content_url_prefix/CODE/ressourceofapackage
 
 Resources
 ---------

Services/ScormEngine.php

@@ -104,7 +104,7 @@ class ScormEngine implements ContainerAwareInterface
 
             if ($course) {
                 $response['code'] = 400;
-                $response['error'] = 'La clef de cours existe déjà. Utilser \'updateCourse\' pour mettre à jour un cours existant';
+                $response['error'] = 'La clef de cours existe déjà. Utiliser \'updateCourse\' pour mettre à jour un cours existant';
                 return $response;
             }