steemit / condenser

The greatest application front-end to the Steem Blockchain.
https://steemit.com
506 stars 429 forks source link

Login failure for passwords which are themselves valid WIF. #2650

Open hernandev opened 6 years ago

hernandev commented 6 years ago

Disclaimer

On this section, SteemConnect will be used to illustrate a behavior that may take place on any other STEEM client, so take SteemConnect ONLY as an example.


Actual behavior

Steem Clients, like SteemConnect which supports the "Create Account with Delegation" functionally, should not, be in charge of password structure.

Since the password structure is not checked, using a valid WIF private key as password, would result in the following logic:

 

At login time, both Condenser and SteemConnect, there's the following flow:

 

Expected behavior

When using a single input field for both password or WIF private key, the provided value, should be treated as password first, then as WIF private key.

OR

More elegantly, Instead of assuming password or WIF private key, and failing after check against a single format, both must be tried.


How to reproduce


Recording Of The Bug

Is not relevant, from the technical point of view to record the screen because this problem is related to underlying logic. The user interface itself behaves like it was intended to.


Reference End-User Solution

While the issue may not be immediately fixed, for people reaching this report while reproducing it, by accident, those are the private key derivation steps need for being able to log in on the created account:

Alternative method: This codepen has SteemJS and you need to replace the values and get the actual account WIF.



Posted on Utopian.io - Rewarding Open Source Contributors

sneak commented 6 years ago

Thank you for this - this is an excellent bug report.