Should DeleteClient return an error if a client does not exist?

[hug-dev]

Currently it's possible to delete a client that does not exist, example with the tool:

[hugdev01@machine parsec-tool]$ ./target/debug/parsec-tool list-clients
[INFO ] No clients in the service.
[hugdev01@machine parsec-tool]$ ./target/debug/parsec-tool delete-client -c "DO_NOT_DELETE_ME🤪"
[INFO ] Client "DO_NOT_DELETE_ME🤪" deleted.

Maybe we should do a list_clients inside the implementation of delete_clients and check if the client requisted does exist before? If not we could return PsaErrorDoesNotExist.

We should also update the page in the book.

[sumitk63]

hey i would like to work on this, can you please assign that issue to me?

[hug-dev]

Hey @sumitk63 !

Awesome, thank you 😃!! Please do not hesitate to ask if you have any questions, here or on our Slack channel.

[hug-dev]

Hey @sumitk63!

Are you still thinking you will have a go at this issue? I am happy to help you in any way I can!