claroline / Claroline

Learning management system
https://www.claroline.com
Other
315 stars 188 forks source link

Refacto des logs #1534

Open LaurentGruber opened 3 years ago

LaurentGruber commented 3 years ago

Log de sécurité (done in #1537)

événement à enregistrer

??? Modération ???

UI

Menu utilisateur > Mon compte : ajouter un onglet sécurité pour afficher les logs de sécurité lié à l'utilisateur.

Administration > tableau de bord : ajouter un onglet sécurité pour arricher l'ensemble des logs de sécurité de la plateforme.

Log fonctionnel

Objectif : Pouvoir suivre la progression des utilisateurs dans une formation (temps passé, score obtenu, élément réalisé,...)

Log opérationnel

Objectif : enregistrer les événement de la plateforme

Log des communications

chalasr commented 3 years ago

Pour ce qui est de revoir le design general, comme vu ensemble:

Elorfin commented 3 years ago

Il faut aussi voir comment on va gérer l'affichage. A l'heure actuel, il y a des templates twig (possibilité d'utiliser des templates custom pour certains logs) qui sont rendus pour chaque log. A vérifier, mais il me semble qu'il y en a plusieurs, un pour l'affichage dans une liste et un pour l'affichage du détails.

Ce qui est lourd. L'intérêt de cette approche étaient de pouvoir afficher une phrase avec un lien intégré.

Pour ce qui est du plugin log, j'aurai mis :

Le but étant de vider ClarolineCoreBundle de tout le code générique d'une application et de conserver uniquement le noyau LMS dans ce bundle. Il est également possible de mettre le code dans ClarolineAppBundle (qui avait été créé pour ça) suivant la quantité de code requise.

A discuter: où est-ce qu'on met la logique. A mon sens, les classes d'event et les listeners associés peuvent résider dans les plugins et sous namespaces existants

D'accord avec ça. A noter qu'il ne devrait plus y avoir de classes d'event liées au logs étant donné qu'on désire directement écouter les events applicatifs.

Elorfin commented 3 years ago

On met des event listeners sur des events métier (claroline)

A discuter : j'aimerai doucement basculer vers de l'event subscribing, c'est peut être l'occasion de commencer. Comme on va devoir retrouver les évènements claroline pour greffer les logs, on pourrait en profiter pour déjà écrire les catalogues d'events des bundles (voir Symfony\Component\HttpKernel\KernelEvents).

chalasr commented 3 years ago

👍 pour utiliser des subscribers plutot que des listeners, perso ça me pose pas de souci de cohérence qu'on ai les deux approches dans le code et qu'on parte sur du subscriber pour les nouveaux, jusqu'au jour où on a le temps de remettre tout d'équerre