Skip to content

/ graylog2-server

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

Jump to bottom

2.0a4 Create user / Full Name required by backend, not by frontend #1885

Closed
SjonHortensius opened this issue Mar 3, 2016 · 2 comments
Closed

2.0a4 Create user / Full Name required by backend, not by frontend #1885

SjonHortensius opened this issue Mar 3, 2016 · 2 comments
Assignees
@edmundoa
Labels
P4 S4 bug triaged web
Milestone
2.0.0

Comments

@SjonHortensius
Copy link

@SjonHortensius SjonHortensius commented Mar 3, 2016

Whey trying to create a new user, unless specifying a Full name, the submit will fail with an unclear error Failed to create user!. It seems to me the frontend can mark this field as required, or the backend could generate a better error. Currently it returns HTTP/1.1 400 Bad Request without a body
@dennisoelkers dennisoelkers self-assigned this Mar 3, 2016
@dennisoelkers dennisoelkers added this to the 2.0.0 milestone Mar 3, 2016
@kroepke kroepke added S4 P4 labels Mar 4, 2016
@edmundoa edmundoa assigned edmundoa and unassigned dennisoelkers Mar 17, 2016
@edmundoa edmundoa closed this in e438784 Mar 17, 2016
@edmundoa edmundoa reopened this Mar 17, 2016
@edmundoa
Copy link
Member

@edmundoa edmundoa commented Mar 17, 2016

This still needs a bit more work, reopening.
edmundoa added a commit that referenced this issue Mar 17, 2016
@edmundoa
Fix user form validations
061132d 
- Check if username is taken
- Check if passwords are long enough
- Check if passwords match

Refs #1885
edmundoa added a commit that referenced this issue Mar 17, 2016
@edmundoa
Fix condition to ask for old password
4729683 
Refs #1885
@edmundoa
Copy link
Member

@edmundoa edmundoa commented Mar 17, 2016

Fixed in e438784, 061132d, and 4729683.
@edmundoa edmundoa closed this Mar 17, 2016
@edmundoa edmundoa mentioned this issue Mar 17, 2016
Open
@kroepke kroepke added triaged and removed triaged labels Sep 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@edmundoa edmundoa

Labels
P4 S4 bug triaged web
Projects
None yet
Milestone
2.0.0
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@dennisoelkers @kroepke @edmundoa @SjonHortensius