Projet

Général

Profil

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

Ajouté par Gwenael Remond il y a environ 8 ans. Mis à jour il y a environ 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Gwenael Remond
Début:
26/01/2016
Echéance:
% réalisé:

100%

Temps estimé:
6.00 h
Temps passé:
Restant à faire (heures):
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.

Révisions associées

Révision a94456bf (diff)
Ajouté par Lionel Morin il y a environ 8 ans

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

Révision 151dea7a (diff)
Ajouté par Lionel Morin il y a environ 8 ans

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

Révision 1c85d8b1 (diff)
Ajouté par Lionel Morin il y a environ 8 ans

Correction import pour le mode debug (ref #14884)

Historique

#1 Mis à jour par Gwenael Remond il y a environ 8 ans

  • Tâche parente mis à #2473

#2 Mis à jour par Daniel Dehennin il y a environ 8 ans

À noter que twisted.python.log est déprécié dans Ubuntu Xenial en faveur de twisted.logger apparu avec twisted 15.2.0.

#3 Mis à jour par Daniel Dehennin il y a environ 8 ans

  • Projet changé de eole-common à ead

#4 Mis à jour par Lionel Morin il y a environ 8 ans

  • % réalisé changé de 0 à 100
  • Restant à faire (heures) changé de 6.0 à 0.5

#5 Mis à jour par Lionel Morin il y a environ 8 ans

Correction avec le commit : 6b383d2b41d064f721c1f9e2bbce74541c0684fd

#6 Mis à jour par Lionel Morin il y a environ 8 ans

  • Statut changé de En cours à Résolu

#7 Mis à jour par Daniel Dehennin il y a environ 8 ans

  • Statut changé de Résolu à Fermé
  • Restant à faire (heures) changé de 0.5 à 0.0

#8 Mis à jour par Lionel Morin il y a environ 8 ans

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

Formats disponibles : Atom PDF