mozilla / node-convict

Featureful configuration management library for Node.js

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

`getSchema()` broken in 6.0.0

MrBartusek opened this issue · comments

getSchema works differently now and migration guide not that;

that code worked before 6.0:

if(!Config.data.getSchema().properties[key]) throw new Error(`'${key}' value don't exist in config schema while something tried to access it`);

now it must be:

if(!Config.data.getSchema()._cvtProperties[key]) throw new Error(`'${key}' value don't exist in config schema while something tried to access it`);

I don't think that's intended

commented

properties was replaced by _cvtProperties to allow the word properties in your config. ;) In 6.0, properties was a private word.

commented

Also 6.0.0 is partial version of wanted changes, because the next version >=7.0 should be have different behavior with getSchema() to do something like that: !Config.data.getSchema()[key]