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

Open j3kz opened this issue 5 years ago • 1 comments

https://github.com/indexzero/nconf/blob/7c3e5944280f34444937ffcd69017e279165bdbc/lib/nconf/common.js#L105

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

Sep 16 '20 15:09 j3kz

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

Dec 11 '20 07:12 yingca1