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.

Sign up for GitHub

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

Jump to bottom

common.merge() uses a Memory instance without wanted options #348

Open

j3kz opened this issue Sep 16, 2020 · 1 comment

j3kz commented Sep 16, 2020

nconf/lib/nconf/common.js

Line 105 in 7c3e594

var store = new Memory();

For example in a file store, common.merge() does not take in account the initial file store options.

yingca1 commented Dec 11, 2020

use a temporary instance will make store.get action missing the options disableDefaultAccessSeparator and accessSeparator effecting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment