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

Default password visibility #511

Closed
eloydegen opened this issue Feb 24, 2020 · 4 comments
Closed

Default password visibility #511

eloydegen opened this issue Feb 24, 2020 · 4 comments

Comments

@eloydegen
Copy link

After I installed QtPass, the passwords from my already existing pass store are visible by default. I'm not sure if this is an intentional choice or a bug, but I don't think there are any other popular password managers who have this behavior. I think the default setting should be changed to hide them by default. Any thoughts about this?

@annejan
Copy link
Member

annejan commented Feb 24, 2020 via email

@eloydegen
Copy link
Author

I just checked KeepassX, it hides your password by default while there is a button next to it to make it one time visible. This feature would make it by default more secure against shoulder surfing. This is not really a problem with very long randomly generated passwords, but quite less with passphrases. Thankfully, there is already an option to hide them, but I think it should be the default.

If this would be set to the default, I think QtPass also need a button because otherwise you would have to copy the password to another application to make it actually visible.

image

@annejan
Copy link
Member

annejan commented Feb 24, 2020

Ahhh . . Yes . . we have that option
Screenshot 2020-02-24 at 12 05 10
Might be smart to make it a default (and add the show button)

@annejan
Copy link
Member

annejan commented Feb 24, 2020

Wizard now has: ui->checkBoxHidePassword->setCheckState(Qt::Checked);
NB: This only works with a clean install (default) of course 😉
Screenshot 2020-02-24 at 16 54 35
But this is there for you when you enable ☑️ Hide Password
Screenshot 2020-02-24 at 16 54 45

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