Skip to content

[TECH] Contextualiser les loggers de Scripts par défaut #11444

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

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

nlepage
Copy link
Member

@nlepage nlepage commented Feb 18, 2025

🥞 Problème

Les logs des scripts ne sont pas contextualisés, à moins que ce soit fait durant le développement du script.

🥓 Proposition

Créer un child logger contextualisé dans le ScriptRunner.

🧃 Remarques

N/A

😋 Pour tester

Exécuter un script quelconque et vérifier que les logs ont bien la propriété event avec le nom de la classe de Script.

@nlepage nlepage added 👀 Tech Review Needed cross-team Toutes les équipes de dev labels Feb 18, 2025
@nlepage nlepage requested a review from bpetetot February 18, 2025 12:16
@nlepage nlepage self-assigned this Feb 18, 2025
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nickel 👍

@pix-service-auto-merge pix-service-auto-merge force-pushed the tech-scripts-contextualized-logger branch from d9e0db5 to f8b740f Compare February 18, 2025 13:16
@pix-service-auto-merge pix-service-auto-merge merged commit 01a7044 into dev Feb 18, 2025
8 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the tech-scripts-contextualized-logger branch February 18, 2025 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants