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

Login as deactivated user shows an error #9978

Closed
RailsViceRoads opened this issue Jun 24, 2018 · 4 comments · Fixed by #10322
Closed

Login as deactivated user shows an error #9978

RailsViceRoads opened this issue Jun 24, 2018 · 4 comments · Fixed by #10322
Assignees
Labels
3. to review Waiting for reviews design Design, UI, UX, etc. papercut Annoying recurring issue with possibly simple fix.
Milestone

Comments

@RailsViceRoads
Copy link

Steps to reproduce

  1. add an user account
  2. deactivate this user
  3. try login for this user on web interface

Expected behaviour

  • The same behaviour as login with wrong user name or
  • the same behaviour as login with wrong password.

Actual behaviour

  • Login with wrong password will work as expected.
  • Login with right password will show an error "user disabled". No login fields displayed. Page reload in web-browser does also not display login screen.

I suspect for privacy reasons login for an deactivated user should work as login for an non existent user.

Server configuration

Nextcloud version: 13.0.4.0

@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #116 (Show login-history to user), #7471 (LDAP User First time login is disabled), #1255 (Force user to change password at first Login), #4759 (Login image not shown in /settings/admin/theming), and #3230 (Sort users by last login date or quota set).

@MariusBluem
Copy link
Member

Uhm .. Do not have a strong opinion on that. If you are logging in with the correct credentials you'd expect any feedback about why you are not able to login - even if it is "disabled", don't you?
If you do not want this as an admin, you just need to disable the user and change his password 😉

Would this work out for you?

@RailsViceRoads
Copy link
Author

Hi Marius, yes this will work. But I think it will be more user friendly to get something like this

grafik

because on the error-page is no navigation item (like a button) to get back to login the page.

grafik

Also I think it is not an "error" if a deactivated user tries an login. But you are right. This is not a bug. :)
Should I close this issue now?

@MorrisJobke MorrisJobke added design Design, UI, UX, etc. papercut Annoying recurring issue with possibly simple fix. and removed bug labels Jul 4, 2018
@MorrisJobke
Copy link
Member

Also I think it is not an "error" if a deactivated user tries an login. But you are right. This is not a bug. :)
Should I close this issue now?

Still something to polish ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews design Design, UI, UX, etc. papercut Annoying recurring issue with possibly simple fix.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants