-
Notifications
You must be signed in to change notification settings - Fork 147
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
Don't cache configs from env #266
Comments
Very useful for testing where you dynamically change config values to assert how the unit-under-test changes accordingly |
If you could provide a failing test (like https://github.com/mozilla/node-convict/blob/master/test/validation-tests.js#L226) it would be easier to fix. |
Is very easy to do without : https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Objets_globaux/Object/assign Object.assign(process.env, { a:1, n:2}); |
Should be closed ? Not reproductible :( |
Could we flag an env-based config so that it is forced to always be read over from then env variable it is tied to? Right now it seems it only reads it upon startup, and keeps it value forever. The use case for this is to change a config during execution.
The text was updated successfully, but these errors were encountered: