Development #36596
paramétrer LOGGED_REQUEST_MAX_SIZE via l'UI de log des connecteurs
0%
Description
Déporter LOGGED_REQUEST_MAX_SIZE en paramétrage du connecteur comme les autres éléments de log.
Fichiers
Demandes liées
Révisions associées
logging: add max sizes to connector log parameters (#36596)
Historique
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Lié à Development #33895: taille des requêtes logguées ajouté
Mis à jour par Frédéric Péters il y a plus de 4 ans
Dans le même lot je suggérerais d'inclure également le paramétrage de la taille max de la réponse.
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0001-logging-add-requests-and-responses-max-size-to-conne.patch 0001-logging-add-requests-and-responses-max-size-to-conne.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ce patch couvre les logs envoyées via ProxyLogger (taille max des requêtes et réponses HTTP).
Je n'ai pas réussi à couvrir les cas où un proxy est passé à utils/__init__.py::log_http_request(). (https://jenkins2.entrouvert.org/job/passerelle-wip/job/wip%252F36596-LOGGED_REQUEST_MAX_SIZE-on-connector-log-parameters/3/cobertura/utils/__init___py/ lignes 162 et 174)
Est-ce que vous aurriez une piste que je puisse suivre ?
Mis à jour par Frédéric Péters il y a plus de 4 ans
Il faut un objet logger attaché à un connecteur, il faut pour ça un connecteur qui utiliser self.requests plutôt que faire des appels directs. (pour le connecteur oxyd que tu utilises pour tester, #21464).
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0001-logging-add-max-sizes-to-connector-log-parameters-36.patch 0001-logging-add-max-sizes-to-connector-log-parameters-36.patch ajouté
via #21464, merci.
Mis à jour par Frédéric Péters il y a plus de 4 ans
Ne pas reprendre le truc bizarre de ne pas avoir la même valeur pour les deux options, tape 5000 par défaut pour les deux.
Idéalement le changement de nom, LOGGED_REQUEST_MAX_SIZE → LOGGED_REQUESTS_MAX_SIZE devrait être dans un commit séparé, si c'est galère, tant pis.
Avec ça, je pense que ça sera ok.
Mis à jour par Nicolas Roche il y a plus de 4 ans
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Statut changé de Solution proposée à Résolu (à déployer)
commit fc77a6e36469cf03417b9147ffb32de555f6e7b1 Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Thu Oct 3 15:59:59 2019 +0200 logging: add max sizes to connector log parameters (#36596) commit f4544e64b9ccc09c69867a793db6b9a70ac598ba Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Fri Oct 25 13:55:29 2019 +0200 logging: rename LOGGED_REQUESTS_MAX_SIZE parameter (#36596)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Nicolas Roche il y a presque 4 ans
- Lié à Bug #42187: Ne pas faire dépendre les tests ProxyLogger d'un connecteur spécifique. ajouté
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Lié à Development #48235: Permettre de suivre un éventuel changement de configuration globale, sur la taille des requêtes et réponses loguées. ajouté
logging: rename LOGGED_REQUESTS_MAX_SIZE parameter (#36596)