Project

General

Profile

Tâche #14884

Distribution EOLE - Scénario #2473: Les logs de logiciel doivent être clairs

Il faut un mode debug logguer les appels xml-rpc dans l'ead

Added by Gwenael Remond over 4 years ago. Updated over 4 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
01/26/2016
Due date:
% Done:

100%

Estimated time:
6.00 h
Spent time:
Remaining (hours):
0.0

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

Revision a94456bf (diff)
Added by Lionel Morin over 4 years ago

Ajout d'un mode debug pour les logs du frontend et du backend EAD (ref #14884 @3h)

Revision 151dea7a (diff)
Added by Lionel Morin over 4 years ago

Suppression d'un print (ref #14884 @0.5h)

Revision 1c85d8b1 (diff)
Added by Lionel Morin over 4 years ago

Correction import pour le mode debug (ref #14884)

History

#1 Updated by Gwenael Remond over 4 years ago

  • Parent task set to #2473

#2 Updated by Daniel Dehennin over 4 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.

#3 Updated by Daniel Dehennin over 4 years ago

  • Project changed from eole-common to ead

#4 Updated by Lionel Morin over 4 years ago

  • % Done changed from 0 to 100
  • Remaining (hours) changed from 6.0 to 0.5

#5 Updated by Lionel Morin over 4 years ago

Correction avec le commit : 6b383d2b41d064f721c1f9e2bbce74541c0684fd

#6 Updated by Lionel Morin over 4 years ago

  • Status changed from En cours to Résolu

#7 Updated by Daniel Dehennin over 4 years ago

  • Status changed from Résolu to Fermé
  • Remaining (hours) changed from 0.5 to 0.0

#8 Updated by Lionel Morin over 4 years ago

Ajout de la variable debug dans les fichiers /usr/share/ead2/backend/config/config.py et /usr/share/ead2/frontend/web/config.py.

Also available in: Atom PDF