electron-push-receiver icon indicating copy to clipboard operation
electron-push-receiver copied to clipboard

Add prefix to `credentials` config item (electron-config)

Open mjarkk opened this issue 7 years ago • 3 comments

After a long time of debugging why a electron app didn't work i found that this package uses electron-config that saves a item named credentials
Because my application also uses electron-config with an saved item named credentials coursed electron-push-receiver to not work with a result that my complete app stopped working.

I think it would be better to add a prefix something like push-receiver-credentials.
I don't think i will be the last one that gets this problem it's also really annoying to debug because the electron error doesn't explain a lot

mjarkk avatar Oct 17 '18 09:10 mjarkk

Went into a similar issue.. the workaround/fix is to not use the default config file provided by electron-config. Use your own settings file by providing your setting file name : const config = new Config("MysettingsFile");

PedroKantar avatar Oct 17 '18 09:10 PedroKantar

Thx for the workaround
I already fixed it in my code by adding a prefix to the things i save in electron-config.

Aside from that this still seems like something that electron-push-receiver needs to handle by specifying another save location or adding a prefix to every item saved with electron-config.
credentials is probably used a lot to save things in electron-config

mjarkk avatar Oct 17 '18 10:10 mjarkk

@mjarkk Feel free to send a PR to fix this

MatthieuLemoine avatar Oct 17 '18 10:10 MatthieuLemoine