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

OCIS Permissions - Failed to edit user #7533

Closed
T0mWz opened this issue Aug 26, 2022 · 3 comments · Fixed by #7538
Closed

OCIS Permissions - Failed to edit user #7533

T0mWz opened this issue Aug 26, 2022 · 3 comments · Fixed by #7538
Assignees
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug Something isn't working

Comments

@T0mWz
Copy link

T0mWz commented Aug 26, 2022

When I have a user without an email address and you try to assign a role or a quota, you get a message Failed to edit user. Maybe an extra hint that a valid emailaddress is required can be useful, now I discover the issue by checking the browser console.

Steps to reproduce

  1. Add a user without an email address
  2. Try to assign a role or some quota

Expected behaviour

Littlebit more information why it is valid. 😉

Actual behaviour

Screenshot 2022-08-26 at 15 52 19

{"error":{"code":"invalidRequest","innererror":{"date":"2022-08-26T13:38:52Z","request-id":"25f743a2-9467-4e8d-9f84-0cde080387de"},"message":"' ' is not a valid email address"}}

Environment general

  • IdP: KeyCloak
  • user storage; LDAP

Environment ownCloud OCIS

  • ownCloud Web UI 5.7.0-rc.10
  • Infinite Scale 2.0.0-beta.7 Community
@AlexAndBear
Copy link
Contributor

AlexAndBear commented Aug 29, 2022

PR #7538
will take care that only the respective endpoints will be called if changes happen for either: user base data, quota, or role.

Anyways, we don't want to allow changes to the user base data (displayName, mail, password) if an external IDP is involved, therefore we plan to make those fields read-only

@T0mWz
Copy link
Author

T0mWz commented Aug 29, 2022

Maybe nice if a link can be configured / shown here , that a user know where this information can be adjusted.

@AlexAndBear
Copy link
Contributor

good idea,
FYI @tbsbdr @kulmann

@kulmann kulmann added Type:Bug Something isn't working Priority:p2-high Escalation, on top of current planning, release blocker labels Aug 29, 2022
@AlexAndBear AlexAndBear self-assigned this Aug 30, 2022
@kulmann kulmann moved this to Done in Web Support Board Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants