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

ETQ operateur, je peux piloter le niveau de log à partir de la variable DS_LOG_LEVEL #9342

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

LeSim
Copy link
Member

@LeSim LeSim commented Jul 21, 2023

Avant le niveau de log était piloté par la variable environnement "DS_ENV".
si DS_ENV == "staging" alors le niveau était degug sinon info.

Maintenant, on peut directement mettre un des niveau dans la variable DS_LOG_LEVEL, les niveaux disponibles sont:
debug, info, warn, error, fatal, and unknown

le défaut est info.

@what-the-diff
Copy link

what-the-diff bot commented Jul 21, 2023

PR Summary

  • Introduction of Flexible Log Levels Configuration
    In technical terms, a line was added to the optional configuration file, assigning the default log level to 'info'. This might seem jargon-heavy, but simply put, it allows the system to record and show the 'info' level activities happening within the system.

  • Improved Production.rb Configuration for Log Levels
    We've also made changes to the production configuration file to use the log level value set by the user (if any) or to default to 'info' level. This change is significant because it provides more flexibility and ease when it comes to managing how much information the system logs depending upon the requirements of the production environment.

Copy link
Contributor

@krichtof krichtof left a comment

Choose a reason for hiding this comment

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

LGTM :)

@LeSim LeSim added this pull request to the merge queue Jul 21, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 21, 2023
@LeSim LeSim added this pull request to the merge queue Jul 21, 2023
Merged via the queue into main with commit aceaf24 Jul 21, 2023
@LeSim LeSim deleted the add_env_to_control_log_level branch July 21, 2023 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants