Skip to content

Add env variable to disable express logging#34

Closed
quentinleclerc wants to merge 4 commits intomendhak:masterfrom
quentinleclerc:add-env-variable-to-disable-express-logging
Closed

Add env variable to disable express logging#34
quentinleclerc wants to merge 4 commits intomendhak:masterfrom
quentinleclerc:add-env-variable-to-disable-express-logging

Conversation

@quentinleclerc
Copy link

Hello,
I've been needing exactly your software but with a slight change so I figured I might as well upload it, if someone needs it.

I just added an environment variable to disable express logging if not needed. I set it to true in the dockerfile to let the default behaviour unchanged.

Thanks,

@mendhak
Copy link
Owner

mendhak commented Jan 3, 2022

I've redone the feature with a different environment variable, going in the opposite direction, I didn't want any extra variables required in the Dockerfile.

Also I've just removed the dashes completely, I don't think it served a good purpose.

I'll close this PR, this is now implemented here: #35

@mendhak mendhak closed this Jan 3, 2022
@mendhak
Copy link
Owner

mendhak commented Jan 6, 2022

OK it's now in tag :23 and works by setting the environment variable, DISABLE_REQUEST_LOGS=true

AAAH I forgot your table! OK I'll add that another time it's a nice useful summary

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