Skip to content
This repository has been archived by the owner on Mar 26, 2021. It is now read-only.

Allow log: Env(LOG_STREST) #49

Closed
jgroom33 opened this issue Oct 11, 2018 · 3 comments
Closed

Allow log: Env(LOG_STREST) #49

jgroom33 opened this issue Oct 11, 2018 · 3 comments

Comments

@jgroom33
Copy link
Collaborator

This would be good to support global logging on/off
log: Env(LOG_STREST)

^ fails schema

@eykrehbein
Copy link
Owner

What do you mean by that exactly?

@adiman9
Copy link
Contributor

adiman9 commented Oct 11, 2018

I believe what is meant is that log currently only accepts a boolean in the schema and it has to be set manually per request in the yaml. It would be good if we could use the Env() operator to use an environment variable to globally turn on / off logging for all requests without having to change the yaml.

@eykrehbein
Copy link
Owner

It was basically already integrated but I forgot that the Joi pattern blocked the use of a string. But this was fixed for the log param in #53 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants