-
Notifications
You must be signed in to change notification settings - Fork 66
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
Password Storage #137
Comments
Just FYI, I'm doing huge refactoring in #138, so you might want to hold off on this task for a couple of days. Otherwise it might be a pain to even merge it :) |
@czechboy0 that's fine :) will do. |
@czechboy0 is this good to go now? |
Yes, the big refactoring has been merged. Feel free to use this issue to track progress and ideas, let's discuss how you'll structure the change before you start so that we consider all aspects of it :) |
FYI, @Brett-Best. I'm taking over this one. I need it done for more work that's queued. |
This has been merged into the v0.8 development branch, so don't expect it in v0.7 yet. Closing to not be in the list of open issues though :) |
Currently credentials are stored in plain text.
Change to use the keychain instead.
The text was updated successfully, but these errors were encountered: