legacy-wallet-use-trinity-wallet-instead
legacy-wallet-use-trinity-wallet-instead copied to clipboard
Misleading tooltip message for seeds of length 60-80 ("Only uppercase letters and the number 9 are allowed")
Prerequisites
If you are not suggesting a feature, you must be able to check all of the following (place an x inside the brackets to check the box). If you cannot check all of the boxes, this issue should not be reported on GitHub as it is not a bug:
- [X] I confirm that this is an issue with the IOTA Wallet and not an exchange
- [X] I confirm that this is an issue with the wallet, not IRI
- [X] For pending transactions: I confirm that I have tried reattaching and promoting multiple times (check this box if your issue is not related to a pending transaction)
- [X] I confirm that this is not an issue related to stolen/lost funds
- [X] I confirm that this is not a zero balance issue related to a snapshot (last snapshots occurred on October 23 and January 28)
- [X] For private key warnings: I confirm that I am not at risk of reusing an address, or another user has verified that I am not (check this box if your issue is not related to a private key reuse)
- [X] I confirm that this issue is present on the latest version of the wallet
Description
When using a seed of length 60 to 80 (so not the full 81 chars), the tooltip on the "!" right of the infobox shows "Only uppercase letters and the number 9 are allowed", yet login is possible and the wallet functional. (When using an invalid seed of that length, the same tooltip appears, but login fails with "Invalid Characters").
Steps to Reproduce
- Open wallet, so you are at the login screen
- Enter seed "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA"
- Observe that the error message complains about invalid characters
- Try to log in (it works)
- Log out again
- Enter seed "42AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA"
- Observe that the error message is the same, yet login fails with "Invalid characters"
Expected behavior: When entering a seed that is too short, the indicator should show a different tooltip (for example a warning that the seed is shorter than 81 characters), and it should not be an "!" but some "info" siymbol (or maybe the checksum like it does with 81 character seeds)
Actual behavior: A misleading error message is shown that makes the user believe his seed is invalid. See for example this reddit thread
Versions
Windows 10 Wallet 2.5.7
No need to fix anything ... Trinity is almost ready and the light wallet will die then :)
I'll bet 10 MIOTA (against the first person who accepts the bet) that the Trinity desktop wallet will not be released (in a stable version) by the end of 2018Q2.