Tâche #14820
Distribution EOLE - Scénario #2473: Les logs de logiciel doivent être clairs
Il faut un mode debug logguer les appels xml-rpc dans controle-vnc
Description
Les applications qui font du xml-rpc comme par exemple controle-vnc font du log très, très détaillé (chaque appel est loggué, avec print des paramètres)
L'exemple du backend controle-vnc d'eop :
xmlrpc_eop.py:317: log.msg('Clé erronnée : %s'%cle) xmlrpc_eop.py:320: log.msg('User non prof : %s'%username) xmlrpc_eop.py:323: log.msg('Appel de la fonction xmlrpc_remote_change_mdp') xmlrpc_eop.py:342: log.msg('Clé erronnée : %s'%cle) xmlrpc_eop.py:345: log.msg('User non prof : %s'%username) xmlrpc_eop.py:348: log.msg('Appel de la fonction xmlrpc_remote_change_mdp') xmlrpc_eop.py:363: log.msg('Clé erronnée : %s'%cle) xmlrpc_eop.py:366: log.msg('User non prof : %s'%username) xmlrpc_eop.py:369: log.msg('Appel de la fonction xmlrpc_password_must_change par %s pour %s' % (username, logineleve)) xmlrpc_eop.py:385: log.msg('Clé erronnée : %s'%cle) xmlrpc_eop.py:388: log.msg('User non prof : %s'%username) xmlrpc_eop.py:391: log.msg('Appel de la fonction xmlrpc_remote_save_password_csv pour %s' % username) xmlrpc_eop.py:424: log.msg('Appel de la fonction remote_get_connectes_groupes par %s pour %s'%(username, group)) xmlrpc_eop.py:426: log.msg('Clé erronnée : %s'%cle) xmlrpc_eop.py:429: log.msg('User non prof : %s'%username) xmlrpc_eop.py:442: log.msg('Appel de la fonction remote_create_vnc_config par %s pour %s'%(username, login)) xmlrpc_eop.py:444: log.msg('Clé erronnée : %s'%cle) xmlrpc_eop.py:447: log.msg('User non prof : %s'%username) xmlrpc_eop.py:477: log.msg('Appel de la fonction remote_delete_vnc_config par %s pour %s'%(username, id)) xmlrpc_eop.py:496: log.msg('Appel de la fonction remote_get_list_profs par %s'%(username)) xmlrpc_eop.py:498: log.msg('Clé erronnée : %s'%cle) xmlrpc_eop.py:501: log.msg('User non prof : %s'%username) xmlrpc_eop.py:533: log.msg('Classe %s non trouvée dans les classes administrées par %s'%(classe, username)) xmlrpc_eop.py:542: log.msg('Appel de la fonction remote_set_admin_right_to_class par %s pour la classe %s et login %s'%(username, classe, login)) xmlrpc_eop.py:544: log.msg('Clé erronnée : %s'%cle) xmlrpc_eop.py:547: log.msg('User non prof : %s'%username) xmlrpc_eop.py:550: log.msg("Impossible de changer les droits de l'admin") xmlrpc_eop.py:553: log.msg("Impossible de se changer les droits à soi-même") xmlrpc_eop.py:570: log.msg(e)
Ces logs peuvent être pertinents et il ne faut pas les effacer, mais il polluent les fichiers de log, et il faudrait les mettre uniquement si on est dans un mode debug.
Par ailleurs, la résolution de #13606 grâce à une directive rsyslog ne supprime pas les logs "POST /RPC2 HTTP/1.1" 200 516 "-" "xmlrpclib.py/1.0.1 (by www.pythonware.com)" de l'EAD.
Étudier aussi l’opportunité de supprimer ou conditionner certains print comme par exemple "Appel à is_defined" dans ead/frontend/web/gabarit.py qui allourdit rapidement les logs.
Associated revisions
ajout d'un mode debug (ref #14820)
Centralisation du debug dans le fichier de config (ref #14820 @1h)
mode debug dans l'ead, ref #14820
History
#1 Updated by Gwenael Remond over 7 years ago
- Tracker changed from Proposition Scénario to Tâche
- Assigned To set to Gwenael Remond
- Estimated time changed from 5.00 h to 12.00 h
- Parent task set to #2473
- Remaining (hours) set to 12.0
#2 Updated by Lionel Morin over 7 years ago
- Description updated (diff)
#3 Updated by Lionel Morin over 7 years ago
- Description updated (diff)
#4 Updated by Scrum Master over 7 years ago
- Status changed from Nouveau to En cours
#5 Updated by Gwenael Remond over 7 years ago
- Subject changed from Il faut un mode debug pour toutes les applications qui font du xml-rpc to Il faut un mode debug logguer les appels xml-rpc dans controle-vnc
- Estimated time changed from 12.00 h to 6.00 h
- Remaining (hours) changed from 12.0 to 6.0
#6 Updated by Daniel Dehennin over 7 years ago
- Project changed from eole-common to controle-vnc
#7 Updated by Daniel Dehennin over 7 years ago
À noter que twisted.python.log est déprécié dans Ubuntu Xenial en faveur de twisted.logger apparu avec twisted 15.2.0.
#8 Updated by Gwenael Remond over 7 years ago
- Remaining (hours) changed from 6.0 to 1.0
#9 Updated by Scrum Master over 7 years ago
- Status changed from En cours to Résolu
#10 Updated by Lionel Morin over 7 years ago
Présence inutile de lignes de log contenant "POST /RPC2 HTTP/1.1" dans /var/log/controle-vnc/main.log.
=> Ré-ouverture de la demande
#11 Updated by Lionel Morin over 7 years ago
- Status changed from Résolu to En cours
#12 Updated by Scrum Master over 7 years ago
- Status changed from En cours to Résolu
#13 Updated by Lionel Morin over 7 years ago
- % Done changed from 0 to 100
- Remaining (hours) changed from 1.0 to 0.5
#14 Updated by Daniel Dehennin over 7 years ago
- Status changed from Résolu to Fermé
- Remaining (hours) changed from 0.5 to 0.0
#15 Updated by Lionel Morin over 7 years ago
Ajout de la variable debug au fichier /usr/share/eole/controlevnc/config.py.