Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installation of user.js causes saved passwords to be removed #27

Closed
crashkopf opened this issue Jul 14, 2015 · 5 comments
Closed

Installation of user.js causes saved passwords to be removed #27

crashkopf opened this issue Jul 14, 2015 · 5 comments

Comments

@crashkopf
Copy link

Installation of user.js causes saved passwords to be removed from the Firefox This wasn't explicitly listed as intended behavior in the README file, though it appears to be quite intentional. I would suggest warning potential users about what may be, to them, an unintended consequence. Strong language about backing up profile data before proceeding would be highly advisable.

Otherwise, great work.

@pyllyukko
Copy link
Owner

Thanks for raising the issue. Have you checked the profile directory, whether the DB file is still there? Or can you try, if the refresh feature can bring them back?

I need to add this to the README.

@crashkopf
Copy link
Author

key3.db and signons.sqlite still exist, but appear to be devoid of any useful data.

I believe the relevant line is:
user_pref("privacy.clearOnShutdown.passwords", true);

The changes from user.js appear to get saved into prefs.js, so I can't save any passwords until I've removed the line about from prefs.js (and removed user.js). Apparently preferences changed in user.js get saved when any changes are comitted via the preferences dialog. See: https://developer.mozilla.org/en-US/docs/Mozilla/Preferences/A_brief_guide_to_Mozilla_preferences

Lucky for me I've been in the habit of making backups lately. ;)

@pyllyukko
Copy link
Owner

Do you think the commit above is sufficient to address this issue?

@crashkopf
Copy link
Author

Yes, I think that will help anyone who reads the instructions thoroughly.

@pyllyukko
Copy link
Owner

Cool. I'll close this issue then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants