Shouldn't re-assign module.exports, better to just set properties on the pre-existing object.

It looks innocuous enough, but it'll eventually bite you in the form of hard to debug require-cycle hilarity (maybe other reasons too).

It's less of an issue for code like this that's a millimeter away from just being a json file, but the point of a settings module is that it can/should be more than just static json.