Skip to content

Allow enabling dev logs#2326

Merged
advplyr merged 1 commit intoadvplyr:masterfrom
lkiesow:hide-dev-logs
Nov 25, 2023
Merged

Allow enabling dev logs#2326
advplyr merged 1 commit intoadvplyr:masterfrom
lkiesow:hide-dev-logs

Conversation

@lkiesow
Copy link
Copy Markdown
Contributor

@lkiesow lkiesow commented Nov 19, 2023

This patch allows users to enable dev logs on production systems by setting the HIDE_DEV_LOGS environment variable.

Before, you could only use this on a non-production environment. On production, the logs would be disabled. This patch changes the behavior and uses the NODE_ENV only as default. On production they are disabled if HIDE_DEV_LOGS is undefined but can be enabled by setting HIDE_DEV_LOGS=0 on dev, they are enabled if undefined, but can be disabled by setting HIDE_DEV_LOGS=1.

This patch allows users to enable dev logs on production systems by
setting the `HIDE_DEV_LOGS` environment variable.

Before, you could only use this on a non-production environment. On
production, the logs would be disabled. This patch changes the behavior
and uses the `NODE_ENV` only as default. On production they are disabled
if `HIDE_DEV_LOGS` is undefined but can be enabled by setting
`HIDE_DEV_LOGS=0` on dev, they are enabled if undefined, but can be
disabled by setting `HIDE_DEV_LOGS=1`.
@advplyr
Copy link
Copy Markdown
Owner

advplyr commented Nov 25, 2023

Works for me, thanks!

@advplyr advplyr merged commit 2e06ae0 into advplyr:master Nov 25, 2023
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