Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TODO : Debug module #30

Open
lowki opened this issue Mar 25, 2013 · 16 comments
Open

TODO : Debug module #30

lowki opened this issue Mar 25, 2013 · 16 comments

Comments

@lowki
Copy link
Contributor

lowki commented Mar 25, 2013

Un joli var_dump des variables GET, POST et des requêtes sur la base.
Une méthode pour ajouter des messages de debug.
Plusieurs niveaux de message (INFO,DEBUG,ERROR)

@wakdev
Copy link
Owner

wakdev commented Mar 25, 2013

J'ai commencé à mettre ça en place : 45761df

A voir si l'implémentation est bonne pour continuer sur cette voie.

@wakdev
Copy link
Owner

wakdev commented Mar 25, 2013

Par contre, je pense que dans l'idéal serai d'utiliser $slash->show_fatal_error et éventuellement une nouvelle méthode $slash->show_error (sans exit) pour afficher l'ensemble des erreurs du CMS, non ?

@wakdev
Copy link
Owner

wakdev commented Mar 25, 2013

  • une méthode permettant l'enregistrement des logs (en bdd ?)

@lowki
Copy link
Contributor Author

lowki commented Mar 26, 2013

Je pense plutôt à un niveau FATAL qui provoquerait un exit.
L'idée c'est d'avoir une console où tous les messages soient rassemblés.
Concernant les logs, j'ai commencé d'y penser un peu, au moins avoir une méthode dans le core qui log tous les accès, pour faciliter le traçage en cas d'attaque.

@wakdev
Copy link
Owner

wakdev commented Mar 26, 2013

La méthode show_fatal_error, est principalement utilisé sur les erreurs de bdd.
Dans ce cas, je pense qu'un exit s'impose.

@lowki
Copy link
Contributor Author

lowki commented Mar 26, 2013

Entièrement d'accord.

Le 26 mars 2013 08:58, wakdev [email protected] a écrit :

La méthode show_fatal_error, est principalement utilisé sur les erreurs de
bdd.
Dans ce cas, je pense qu'un exit s'impose.


Reply to this email directly or view it on GitHubhttps://github.com//issues/30#issuecomment-15445593
.

Loïc BAJARD
53 rue Faubourg Madeleine, 21200 Beaune
09 52 44 06 25
06 84 56 95 57

@wakdev
Copy link
Owner

wakdev commented Mar 27, 2013

Ce soir, si j'ai le temps, j'ajoute l'implémentation des logs :

Une table sl_logs :
id, log_type, url, info, user, date

Dans le fichier de configuration, ajout de deux variables :
$logs = true ou false;
$logs_rotation = "week";

Dans le noyau slash ajout de deux méthodes :
log($info,$log_type=LOG_TYPE_INFO) et clear_log();
log() permet de récupérer automatiquement l'url, l'utilisateur, la date et fait une rotation si nécessaire.

Ajout des constantes LOG_TYPE_* pour les constantes des types de log

Plus tard un module sl_logs sera disponible pour visualiser la liste des logs.

@lowki lowki closed this as completed Mar 27, 2013
@lowki lowki reopened this Mar 27, 2013
@lowki
Copy link
Contributor Author

lowki commented Mar 27, 2013

C'est prometteur. Bon courage.

@wakdev
Copy link
Owner

wakdev commented Mar 27, 2013

Enjoy : bcf8dfd

Utilisation dans les modules : $this->slash->("log");
Ou plus complet : $this->slash->("log","titre",CONSTANT TYPE LOG);

Pour plus de lisibilité, je conseil de placer dans le titre le nom du module en cours.
L'activation des logs et paramétrage de la rotation est situé dans le fichier de config.

@wakdev
Copy link
Owner

wakdev commented Mar 27, 2013

Sur conseil, il serai bien aussi de stocker l'url referer.
Je vois pour l'intégrer plus tard.

@wakdev
Copy link
Owner

wakdev commented Mar 28, 2013

Je vais voir aussi pour intégrer les logs dans des modules existants.

@wakdev
Copy link
Owner

wakdev commented Mar 30, 2013

URL REFERER ajouté : a6559ca

@wakdev
Copy link
Owner

wakdev commented Mar 30, 2013

Reste l'intégration des logs dans les différents modules

@wakdev
Copy link
Owner

wakdev commented Apr 26, 2013

Une première version du module admin permettant la visualisation des logs a été ajouté : 1d79052

@lowki
Copy link
Contributor Author

lowki commented Apr 26, 2013

Wouhou j'ai hâte de tester ça

@wakdev
Copy link
Owner

wakdev commented Apr 26, 2013

Il n'y a pas grand chose, c'est juste pour avoir un début de module ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants