-
-
Notifications
You must be signed in to change notification settings - Fork 121
Support _FILE based configuration values #264
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
Conversation
|
@m90 used mostly your code from #253 for the lookup function and pointed the I also experimented with the Edit: The |
|
This looks like a great addition, thanks.
Ok, just let me know when you're done waiting. Or not.
Good idea, I'll add these to #261 |
|
Created the PR to the source repo: kelseyhightower/envconfig#210 |
|
Done from my side :) |
m90
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
Closes #253
Changes
All environment variables can now be used with the
_FILEconvention.This means, that e.g.
WEBDAV_PASSWORDcan also be supplied via docker secrets by usingWEBDAV_PASSWORD_FILEinstead.Note: All value (non-file) variants are preferred over their _FILE variant.
Todo