Accepts empty password on new wallet and account creation
manan-m-shah opened this issue · 3 comments
manan-m-shah commented
Not sure if it's intentional, but the commands below are accepting empty passwords.
forc wallet new
forc wallet account new
Can quickly patch this if required.
kayagokalp commented
I think we can disallow empty passwords as a security measure. cc @FuelLabs/tooling for ideatation
kayagokalp commented
@manan-m-shah are you still interested in implementing this? I think we want this implemented, and will likely look into it in coming weeks. If you want to take a shot, go for it!
sdankel commented
I agree, this would be a good security measure. This code changes will be in https://github.com/FuelLabs/forc-wallet