node-sync-files icon indicating copy to clipboard operation
node-sync-files copied to clipboard

TypeError: Cannot read property 'optOut' of undefined

Open AndyOGo opened this issue 8 years ago • 8 comments

The configstore randomly throws this error.

.../node_modules/sync-files/node_modules/configstore/index.js:86
    return this.all[key];
                   ^
TypeError: Cannot read property 'optOut' of undefined
    at Object.Configstore.get (.../node_modules/sync-files/node_modules/configstore/index.js:86:17)
    at UpdateNotifier.check (.../node_modules/sync-files/node_modules/update-notifier/index.js:47:18)
    at module.exports (.../node_modules/sync-files/node_modules/update-notifier/index.js:127:17)
    at Object.<anonymous> (.../node_modules/sync-files/bin/sync-files.js:121:3)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Function.Module.runMain (module.js:447:10)
    at startup (node.js:146:18)

AndyOGo avatar Oct 05 '16 11:10 AndyOGo

This bug can be "disabled" by setting --no-notify-update

AndyOGo avatar Oct 21 '16 08:10 AndyOGo

I have the same issue. It only happens once in a while though. What does --no-notify-update mean?

wamoyo avatar Oct 25 '18 23:10 wamoyo

Please see the usage secrion of the readme https://github.com/byteclubfr/node-sync-files/blob/master/README.md#usage

AndyOGo avatar Oct 26 '18 05:10 AndyOGo

Right, that's the help text. But I don't understand the relationship between that option and this error?

wamoyo avatar Oct 26 '18 05:10 wamoyo

It's a great module, by the way! Hasn't given me any other problems ever : ) Even this occasional error messages isn't really a problem, just kind of confusing. Can't tell why it's coming up, or why it only happens sometimes.

wamoyo avatar Oct 26 '18 05:10 wamoyo

Okay. The stack trace above tells you where it comes from. Or just debug it if you want to know, https://github.com/byteclubfr/node-sync-files/blob/master/bin/sync-files.js#L120-L122

AndyOGo avatar Oct 26 '18 05:10 AndyOGo

btw update-notifier is another package https://github.com/yeoman/update-notifier

AndyOGo avatar Oct 26 '18 05:10 AndyOGo

Thank, man!

wamoyo avatar Oct 26 '18 16:10 wamoyo