Skip to content
This repository has been archived by the owner on Oct 8, 2020. It is now read-only.

Show password #148

Open
DevObs1 opened this issue May 3, 2018 · 6 comments
Open

Show password #148

DevObs1 opened this issue May 3, 2018 · 6 comments

Comments

@DevObs1
Copy link
Contributor

DevObs1 commented May 3, 2018

8d91373

This commit is not consistent with the alerts in the registration screen.

We indicate to the user that no one can help him recover the password and that it must be written on paper.
image

And then we add a new button to display his password. (/account)
image

@lazarovicedo
Copy link
Contributor

I disagree..
1 - All 3 statements at the login page are true, no one can help you to recover de password.
2- If you can click on "Show password" is because you are logged-in, and if you are logged-in that is because you knew your password to login.

@daivy
Copy link
Contributor

daivy commented May 4, 2018

I have never seen a website where you can see your password in plain text, sounds weird and unsafe to me.

@lazarovicedo

1 - All 3 statements at the login page are true, no one can help you to recover de password.

And then we implement a way to recover the password?

2- If you can click on "Show password" is because you are logged-in, and if you are logged-in that is because you knew your password to login.

Or you just so happened to walk past someone else's account and can view the password with the click of a button.

@DevObs1
Copy link
Contributor Author

DevObs1 commented May 4, 2018

@lazarovicedo

1 - All 3 statements at the login page are true, no one can help you to recover de password.

They are not at once, since there is someone who maintains this website and has allowed (via this feature) to recover the password.

2- If you can click on "Show password" is because you are logged-in, and if you are logged-in that is because you knew your password to login.

If I'm already connected it's because I know my password, I do not need to be shown.
At the security level, it's average.

I confirm what @daivyy mentioned, never seen a website where you can see your password in plain text.
At best, you can reset it.

@funoverip
Copy link

If you click on "Show password", it doesn't send any request to the website actually. The password is not received from the network. The password is known by the web browser because it is locally stored during login (Try with BurpSuite or any local proxy application.)

screenshot 2018-05-04_14-35-26

But I agree that it is confusing for the end-user, and that this feature should be removed on the mainnet.

@lazarovicedo
Copy link
Contributor

you are right, the show password feature could be removed

@lazarovicedo
Copy link
Contributor

PR #153 removes the show password button

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

No branches or pull requests

4 participants