Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blank Password is accepted in the create command #169

Open
nus-se-bot opened this issue Nov 3, 2018 · 0 comments
Open

Blank Password is accepted in the create command #169

nus-se-bot opened this issue Nov 3, 2018 · 0 comments
Assignees
Labels
severity.High A flaw that affects most users and causes major problems for users. i.e., makes the product almost u type.Bug A bug

Comments

@nus-se-bot
Copy link

When using the create command to create an account, the command
create u/abcde pw/ will run, which is undocumented behaviour (accepting empty string as password). If passwords should be optional, maybe it would be better to indicate usage as create u/USERNAME [pw/PASSWORD], since the password field is actually optional, and blank passwords is not in the spirit of having a password field.


Reported by: @zioul123
Severity: Low

[original: nus-cs2103-AY1819S1/pe-dry-run#142]

@aspiringdevslog aspiringdevslog self-assigned this Nov 4, 2018
@aspiringdevslog aspiringdevslog added type.Bug A bug severity.High A flaw that affects most users and causes major problems for users. i.e., makes the product almost u labels Nov 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity.High A flaw that affects most users and causes major problems for users. i.e., makes the product almost u type.Bug A bug
Projects
None yet
Development

No branches or pull requests

2 participants