Changeset 100609 in spip-zone for _plugins_/migrateur/trunk


Ignore:
Timestamp:
Nov 21, 2016, 6:12:41 PM (2 years ago)
Author:
marcimat@…
Message:

Afficher les erreurs un peu plus visibles dans les logs…

Location:
_plugins_/migrateur/trunk
Files:
9 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/migrateur/trunk/class/Client.php

    r100605 r100609  
    2727        public function action($action, $args = null) {
    2828                if (!is_string($action)) {
    29                         $this->log("Action incomprise");
     29                        $this->warning("Action incomprise");
    3030                        return false;
    3131                }
     
    3434
    3535                if (!class_exists($ActionClass)) {
    36                         $this->log("Action inconnue");
     36                        $this->warning("Action inconnue");
    3737                        return false;
    3838                }
     
    7171        private function analyser_reponse($reponse, $decrypt = true) {
    7272                if (!$reponse) {
    73                         $this->log("Réponse en échec");
     73                        $this->warning("Réponse en échec");
    7474                        return false;
    7575                }
    7676
    7777                if (!is_array($reponse)) {
    78                         $this->log("Type de réponse erroné");
     78                        $this->warning("Type de réponse erroné");
    7979                        var_dump($reponse);
    8080                        #print_r($reponse);
     
    8383
    8484                if (isset($reponse['error'])) {
    85                         $this->log("Retour d'erreur indiqué");
    86                         $this->log($reponse['error']);
     85                        $this->warning("Retour d'erreur indiqué");
     86                        $this->error($reponse['error']);
    8787                        return false;
    8888                }
    8989
    9090                if (!isset($reponse['message'])) {
    91                         $this->log("Retour incompris ou absence de message");
     91                        $this->warning("Retour incompris ou absence de message");
    9292                        return false;
    9393                }
     
    146146        private function prepare_json_request_message($action, $data) {
    147147                if (!$this->auth_key OR !$this->url_source) {
    148                         $this->log("Clé d'authentification ou URL du site source non défini");
     148                        $this->warning("Clé d'authentification ou URL du site source non défini");
    149149                        return false;
    150150                }
     
    191191                $code = curl_getinfo($curl, CURLINFO_HTTP_CODE);
    192192                if ($code != 200) {
    193                         $this->log("Réponse erronée du serveur. Code d'erreur : $code // " . curl_error($curl));
     193                        $this->warning("Réponse erronée du serveur. Code d'erreur : $code // " . curl_error($curl));
    194194                        throw new \RuntimeException('HTTP error: ' . $code . " // " . curl_error($curl));
    195195                }
     
    243243                if (!in_array('crypteur.decrypt', stream_get_filters())) {
    244244                        if (!stream_filter_register('crypteur.decrypt', '\SPIP\Migrateur\Crypteur\DecryptFilter')) {
    245                                 $this->log("Filtre de decryptage introuvable");
     245                                $this->error("Filtre de decryptage introuvable");
    246246                                return false;
    247247                        }
     
    253253
    254254                if (!$fp) {
    255                         $this->log("Serveur source indisponible.");
    256                         return false;
    257                 }
    258 
    259                 stream_filter_append($fp, 'crypteur.decrypt', STREAM_FILTER_READ, array('crypteur' => $this->crypteur));
     255                        $this->warning("Serveur source indisponible.");
     256                        return false;
     257                }
     258
     259
     260                if (!@stream_filter_append($fp, 'crypteur.decrypt', STREAM_FILTER_READ, array('crypteur' => $this->crypteur))) {
     261                        $this->error('Décrypteur inutilisable : ' . error_get_last()['message']);
     262                        fclose($fp);
     263                        return false;
     264                }
     265
    260266                file_put_contents($chemin, $fp);
    261267
     
    303309                }
    304310        }
     311
     312        /**
     313         * Log an error message
     314         *
     315         * @param string message
     316         */
     317        public function error($message)
     318        {
     319                if ($this->logger) {
     320                        $this->logger->error($message);
     321                }
     322        }
     323
     324        /**
     325         * Log a warning message
     326         *
     327         * @param string message
     328         */
     329        public function warning($message)
     330        {
     331                if ($this->logger) {
     332                        $this->logger->warning($message);
     333                }
     334        }
    305335}
  • _plugins_/migrateur/trunk/class/Client/Action/GetFile.php

    r88464 r100609  
    2424                if ($reponse) {
    2525                        if ($hash !== $reponse['message']['data']['hash']) {
    26                                 $this->log("Hash différents, suppression du fichier par sécurité");
     26                                $this->error("Hash différents, suppression du fichier par sécurité");
    2727                                $chemin = $this->destination->dir . DIRECTORY_SEPARATOR . $file;
    2828                                unlink($chemin);
  • _plugins_/migrateur/trunk/class/Client/Action/SyncDirectory.php

    r88705 r100609  
    186186
    187187                        if (!$reponse) {
    188                                 migrateur_log("Échec de récupération du fichier");
     188                                migrateur_log_error("Échec de récupération du fichier");
    189189                                return false;
    190190                        }
  • _plugins_/migrateur/trunk/class/Client/Log.php

    r88450 r100609  
    141141                        $done = true;
    142142                }
     143                switch ($level) {
     144                        case Log::WARNING:
     145                                $message = '<warning>' . $message . '</warning>';
     146                                break;
     147                        case Log::ERROR:
     148                                $message = '<error>' . $message . '</error>';
     149                                break;
     150                }
    143151                return migrateur_log($message);
    144152        }
  • _plugins_/migrateur/trunk/class/Serveur/Action/ActionBase.php

    r88450 r100609  
    7474
    7575        /**
     76         * Log an error message
     77         *
     78         * @param string message
     79         */
     80        public function error($message)
     81        {
     82                if ($this->logger) {
     83                        $this->logger->error($message);
     84                }
     85        }
     86
     87        /**
     88         * Log a warning message
     89         *
     90         * @param string message
     91         */
     92        public function warning($message)
     93        {
     94                if ($this->logger) {
     95                        $this->logger->warning($message);
     96                }
     97        }
     98
     99        /**
    76100         * Log a start run message
    77101         *
  • _plugins_/migrateur/trunk/inc/migrateur.php

    r100604 r100609  
    5454
    5555
    56 
     56/**
     57 * Loge une erreur
     58 *
     59 * @param string $msg
     60**/
     61function migrateur_log_error($msg) {
     62        migrateur_log('<error>' . $msg . '</error>');
     63}
    5764
    5865/**
  • _plugins_/migrateur/trunk/migrateur/mig_bdd_source_make_and_get_dump_sql.php

    r88474 r100609  
    1313
    1414        if (!$reponse) {
    15                 migrateur_log("Échec de la sauvegarde");
     15                migrateur_log_error("Échec de la sauvegarde");
    1616                return;
    1717        }
     
    3232
    3333        if (!$reponse) {
    34                 migrateur_log("Échec de récupération du fichier");
     34                migrateur_log_error("Échec de récupération du fichier");
    3535                return;
    3636        }
  • _plugins_/migrateur/trunk/prive/squelettes/inclure/migrateur-destination.html

    r100606 r100609  
    4848#logs_last pre code, #logs_now pre code { color:#A37247; }
    4949#logs_last pre em, #logs_now pre em { color:#1490CB; font-style:normal; }
     50/* balises non html… mais pratique dans ce cas là */
     51#logs_last pre error, #logs_now pre error { color:#D60500; }
     52#logs_last pre warning, #logs_now pre warning { color:#E29800; }
    5053#logs_now { display:none; }
    5154#logs_now.actif .inner .hd { background-color: #fce201; }
     
    145148                                                }
    146149                                                this_response = $.trim(this_response);
     150                                                console.log(this_response)
    147151                                                if (this_response.length) {
    148152                                                        var lignes = this_response.match(/\[^\r\n\]+/g);
     
    150154                                                                ligne = $.trim(ligne);
    151155                                                                var message = $.parseJSON(ligne);
    152                                                                 $log.append(message.log);
     156                                                                if (message) {
     157                                                                        $log.append(message.log);
     158                                                                }
    153159                                                        });
    154160                                                }
  • _plugins_/migrateur/trunk/prive/squelettes/inclure/migrateur-source.html

    r90816 r100609  
    2222#logs code { color:#A37247; }
    2323#logs em { color:#1490CB; font-style:normal; }
     24/* balises non html… mais pratique dans ce cas là */
     25#logs error { color:#D60500; }
     26#logs warning { color:#E29800; }
    2427</style>
    2528
Note: See TracChangeset for help on using the changeset viewer.