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

Cannot update user information #2528

Closed
edmundoa opened this issue Jul 25, 2016 · 1 comment
Closed

Cannot update user information #2528

edmundoa opened this issue Jul 25, 2016 · 1 comment
Assignees
Labels
Milestone

Comments

@edmundoa
Copy link
Member

@edmundoa edmundoa commented Jul 25, 2016

Problem description

Logged in as a user with the admin role, I cannot edit my information. When changing the form and clicking on "Update user", I get a 400 error and this error in the browser's console:

There was an error fetching a resource: cannot PUT http://127.0.0.1:12900/users/hans (400). Additional information: Unable to map property read_only.
Known properties include: startpage, permissions, timezone, full_name, email, session_timeout_ms, roles

Nothing else is shown on the server logs.

Steps to reproduce the problem

  1. Log in as a user with the admin role
  2. Go to Your user -> Edit profile
  3. Change some information, i.e. time zone
  4. Press on "Update user" and see the error

Environment

  • Graylog Version: 2.1.0-alpha.3 (d189efd)
  • Browser version: Chrome 52
@edmundoa edmundoa added the bug label Jul 25, 2016
@edmundoa edmundoa added this to the 2.1.0 milestone Jul 25, 2016
@kroepke kroepke self-assigned this Jul 26, 2016
kroepke added a commit that referenced this issue Jul 26, 2016
 - use better overflow when users have many roles, fixes #2526
 - ignore non-updatable properties in ChangeUserRequest fixes #2528
 - show session information in the user list fixes #2540
@bernd bernd added the to-verify label Aug 1, 2016
@bernd
Copy link
Member

@bernd bernd commented Aug 1, 2016

Please check if this is still an issue, thanks!

edmundoa added a commit that referenced this issue Aug 1, 2016
#2541)

* display session information, fix usability in user list, editing users

 - use better overflow when users have many roles, fixes #2526
 - ignore non-updatable properties in ChangeUserRequest fixes #2528
 - show session information in the user list fixes #2540

* move inline styles into separate component css file

* check for permissions when rendering edit user form

in order to allow reader users to change their details we need to avoid redirecting to a page they cannot see
also add new permission for changing authentication settings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants