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

Fichier spécifique pour les log d'autorisation #16

Closed
wants to merge 3 commits into from
Closed

Fichier spécifique pour les log d'autorisation #16

wants to merge 3 commits into from

Conversation

phenix-factory
Copy link

Afin de faciliter le débug et d'échapper au bruit qu'il y a dans spip.log

Afin de faciliter le débug et d'échapper au bruit qu'il y a dans spip.log
@brunob
Copy link
Contributor

brunob commented Jun 24, 2015

Bonne idée, à discuter. Par contre il manque la même modif à la ligne 195 du même fichier.

@phenix-factory
Copy link
Author

My bad, c'est corrigé.

@phenix-factory
Copy link
Author

Qu'est-ce qu'il y a discuter la dessus ? Je ne vois rien de négatif à ranger des les logs.

@brunob
Copy link
Contributor

brunob commented Jun 30, 2015

Qu'est-ce qu'il y a discuter la dessus ?

Quand je dis qu'il faut en discuter, c'est tout simplement que j'estime qu'il est bien d'en discuter aedc l'équipe avant de merger :)

@Cerdic
Copy link
Contributor

Cerdic commented Jun 30, 2015

Notamment, je pense que ce _DEBUG_AUTORISER n'a plus de sens, il etait là pour ne pas polluer les logs et ne l'activer que quand c'est nécessaire. un simple
spip_log(...,'autoriser'._LOG_DEBUG);
partout serait aussi bien car cela eviterait de devoir aller faire un define specifique.

au profit de l'utilisation des niveaux de log (_LOG_DEBUG)
@phenix-factory
Copy link
Author

Une autre bonne idée, j'ai mis à jour la pull request :)

@Cerdic
Copy link
Contributor

Cerdic commented Jul 24, 2015

C'est intégré manuellement https://core.spip.net/projects/spip/repository/revisions/22328

@Cerdic Cerdic closed this Jul 24, 2015
Cerdic added a commit that referenced this pull request Jul 24, 2015
…de log _LOG_DEBUG. La constante _DEBUG_AUTORISER disparait (Didier via PR #16)
camlafit pushed a commit that referenced this pull request Jul 30, 2015
…de log _LOG_DEBUG. La constante _DEBUG_AUTORISER disparait (Didier via PR #16)
Cerdic added a commit that referenced this pull request Jul 31, 2015
…de log _LOG_DEBUG. La constante _DEBUG_AUTORISER disparait (Didier via PR #16)
Cerdic added a commit that referenced this pull request Aug 1, 2015
…de log _LOG_DEBUG. La constante _DEBUG_AUTORISER disparait (Didier via PR #16)
Cerdic added a commit that referenced this pull request Aug 1, 2015
…de log _LOG_DEBUG. La constante _DEBUG_AUTORISER disparait (Didier via PR #16)
Cerdic added a commit that referenced this pull request Aug 4, 2015
…de log _LOG_DEBUG. La constante _DEBUG_AUTORISER disparait (Didier via PR #16)
Cerdic added a commit that referenced this pull request Aug 5, 2015
…de log _LOG_DEBUG. La constante _DEBUG_AUTORISER disparait (Didier via PR #16)
@phenix-factory phenix-factory deleted the patch-4 branch August 9, 2015 21:46
JamesRezo pushed a commit to spip-remix/bridge-security that referenced this pull request Apr 9, 2024
…de log _LOG_DEBUG. La constante _DEBUG_AUTORISER disparait (Didier via PR spip/SPIP#16)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants