Validate the command args before ask for the master password #13

New issue

Closed

opened 2024-04-24 09:59:58 +02:00 by awiteb · 0 comments

awiteb commented

2024-04-24 09:59:58 +02:00

Owner

Feature description

Sometimes the entered command args is kindly wrong, for example in the add command you can't create an empty vault

https://git.4rs.nl/awiteb/lprs/src/branch/master/src/cli/add_command.rs#L33-L39

So, I think we must validate the args before ask for the master password here (we should ask for the master password when we are sure that the command will run successfully)

https://git.4rs.nl/awiteb/lprs/src/branch/master/src/cli/mod.rs#L61-L67

## Feature description Sometimes the entered command args is kindly wrong, for example in the `add` command you can't create an empty vault https://git.4rs.nl/awiteb/lprs/src/branch/master/src/cli/add_command.rs#L33-L39 So, I think we must validate the args before ask for the master password here (we should ask for the master password when we are sure that the command will run successfully) https://git.4rs.nl/awiteb/lprs/src/branch/master/src/cli/mod.rs#L61-L67

awiteb added the

labels 2024-04-24 09:59:58 +02:00

awiteb changed title from ~~[Feature request]: Validate the command args before ask for the master password~~ to Validate the command args before ask for the master password

2024-04-24 10:19:15 +02:00

awiteb referenced this issue from a pull request that will close it,

2024-04-29 13:36:16 +02:00

feat: Validate args before ask for the master password #17

awiteb closed this issue

2024-04-29 13:39:37 +02:00

awiteb referenced this issue from a commit

2024-04-29 13:39:39 +02:00

feat: Validate args before ask for the master password (#17)

awiteb added this to the v2.0.0 milestone 2024-05-10 13:07:51 +02:00