Project

General

Profile

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

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 d4b9d733 (diff)
Added by Gwenael Remond over 4 years ago

ajout d'un mode debug (ref #14820)

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

Centralisation du debug dans le fichier de config (ref #14820 @1h)

Revision b1089c0a (diff)
Added by Gwenael Remond over 4 years ago

mode debug dans l'ead, ref #14820

History

#1 Updated by Gwenael Remond over 4 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 4 years ago

  • Description updated (diff)

#3 Updated by Lionel Morin over 4 years ago

  • Description updated (diff)

#4 Updated by Scrum Master over 4 years ago

  • Status changed from Nouveau to En cours

#5 Updated by Gwenael Remond over 4 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 4 years ago

  • Project changed from eole-common to controle-vnc

#7 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.

#8 Updated by Gwenael Remond over 4 years ago

  • Remaining (hours) changed from 6.0 to 1.0

#9 Updated by Scrum Master over 4 years ago

  • Status changed from En cours to Résolu

#10 Updated by Lionel Morin over 4 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 4 years ago

  • Status changed from Résolu to En cours

#12 Updated by Scrum Master over 4 years ago

  • Status changed from En cours to Résolu

#13 Updated by Lionel Morin over 4 years ago

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

#14 Updated by Daniel Dehennin over 4 years ago

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

#15 Updated by Lionel Morin over 4 years ago

Ajout de la variable debug au fichier /usr/share/eole/controlevnc/config.py.

Also available in: Atom PDF