-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
Possible to allow http debug to be set via environment variable? #496
Comments
Your report is appreciated. Please star this repository to motivate its developers! ⭐ |
Your report is appreciated. Please star this repository to motivate its developers! ⭐ |
please check how the viper supports ENV github.com/spf13/viper |
@adubovikov thanks, But homer-app currently doesn't use viper to get environment variables only to read the config as far as I can tell . I was referring to this: https://github.com/sipcapture/homer-app/blob/master/docker/docker-entrypoint.d/1 |
I know and I wrote this comment because you mentioned PR :-) otherwise we
will redesign the part of viper reader to use it in more intelligent way.
…On Thu, Oct 27, 2022, 1:02 AM Anthony Bible ***@***.***> wrote:
@adubovikov <https://github.com/adubovikov> thanks, But homer-app
currently doesn't use viper to get environment variables only to read the
config as far as I can tell . I was referring to this:
https://github.com/sipcapture/homer-app/blob/master/docker/docker-entrypoint.d/1
—
Reply to this email directly, view it on GitHub
<#496 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABCN2JO2344FI7T42QD2GTTWFG2BFANCNFSM6AAAAAARPJIRBQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
We have started working on it. |
It would be nice to allow the http debug to be set by an environment variable at startup. Is there a reason it isn't? If not I'd be more than willing to create a PR to allow that.
The text was updated successfully, but these errors were encountered: