TheThing/nconf-lite
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Adding helpful information in case parsing failed.

Browse source 
I stumbled over 2 slight problems of broken configuration files:

1) It said that a error in my JSON file existed (even though it was a yaml file, parsed by libyaml)
2) It didn't tell me which error occured.

both should be fixed with this PR
This commit is contained in:
Martin Heidegger 2014-03-09 21:40:10 +09:00
parent 8105c761ad
commit c8b6c98c7a
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/nconf/stores/file.js
View file

@ -105,7 +105,7 @@ File.prototype.load = function (callback) {
} }
catch (ex) { catch (ex) {
return callback(new Error("Error parsing your JSON configuration file: [" + self.file + '].')); return callback(new Error("Error parsing your configuration file: [" + self.file + ']: ' + ex.message));
} }
callback(null, self.store); callback(null, self.store);
@ -138,7 +138,7 @@ File.prototype.loadSync = function () {
this.store = data; this.store = data;
} }
catch (ex) { catch (ex) {
throw new Error("Error parsing your JSON configuration file: [" + self.file + '].'); throw new Error("Error parsing your configuration file: [" + self.file + ']: ' + ex.message);
} }
} }