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.

Sign up for GitHub

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

Jump to bottom

disable save-passwords #75

Closed
selvanair opened this issue Jan 16, 2017 · 4 comments
Closed

disable save-passwords #75

selvanair opened this issue Jan 16, 2017 · 4 comments

Comments

@selvanair
Copy link
Contributor

With this feature proposed for the GUI, shall we add the registry key during install (set to 0, i.e., save password enabled by default) and also provide an option to set it to 1 (i.e., save password disabled) ?

Adding the key at install time has advantage that admins need not consult README to see how exactly to spell the key. Providing an option to set/unset during install is also nice, but it adds one more section (though hidden under the Advanced options).

Thoughts?
@chipitsine
Copy link
Contributor

Installer option?

@mattock
Copy link
Member

mattock commented Jan 17, 2017

An installer option / new section would be easier from deployment perspective for admins who don't use configuration management tools (=the large majority).

@selvanair selvanair mentioned this issue Jan 17, 2017
Merged
@chipitsine
Copy link
Contributor

this issue might be closed after PR#77 ?

@selvanair
Copy link
Contributor Author

Implemented in PR #77

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mattock @chipitsine @selvanair