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.

Sign up for GitHub

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

Jump to bottom

BUG: creating a user twice via the CLI should error obviously #3576

Open
brassy-endomorph opened this issue Jan 26, 2024 · 1 comment
Open

BUG: creating a user twice via the CLI should error obviously #3576

brassy-endomorph opened this issue Jan 26, 2024 · 1 comment
Labels
bug Things that should work, but don’t Minor

Comments

@brassy-endomorph
Copy link
Contributor

brassy-endomorph commented Jan 26, 2024 •
edited

Describe the bug

To Reproduce

Do this:

aleph create user test@test.com -p abc123
aleph create user test@test.com -p def456

Expected behavior

There is an error of any sort, and the password is not altered.

Aleph version

3.15.3
@brassy-endomorph brassy-endomorph added bug Things that should work, but don’t triage These issues need to be reviewed by the Aleph team labels Jan 26, 2024
@stchris
Copy link
Contributor

stchris commented Feb 13, 2024

The command create user is actually an UPSERT operation, so create user should fail if called on an existing user and we should have a proper update user command which does the above.

@stchris stchris added Minor and removed triage These issues need to be reviewed by the Aleph team labels Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Things that should work, but don’t Minor
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@stchris @brassy-endomorph