Skip to content
This repository has been archived by the owner on Jun 25, 2021. It is now read-only.

simple_key_value_store input validation lacking #314

Closed
Fraser999 opened this issue Jun 10, 2015 · 0 comments
Closed

simple_key_value_store input validation lacking #314

Fraser999 opened this issue Jun 10, 2015 · 0 comments
Assignees

Comments

@Fraser999
Copy link
Contributor

If the following invalid input is entered to the simple_key_value_store example it results in a crash:

get
put
put 1

If put 1 1 1 is attempted, it is treated as put 1 1.

I highly recommend (ab)using docopt for this in the same way as has been done for the crust_peer example which initially suffered from identical problems:
https://github.com/Fraser999/crust/blob/master/examples/crust_peer.rs#L77-L92
https://github.com/Fraser999/crust/blob/master/examples/crust_peer.rs#L432-L504

@Fraser999 Fraser999 self-assigned this Jun 10, 2015
rossmuir pushed a commit that referenced this issue Jun 11, 2015
maqi pushed a commit to maqi/routing that referenced this issue Oct 13, 2015
maqi pushed a commit to maqi/routing that referenced this issue Oct 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant