Bug 51001 - Improve usability if password self reset is disabled
Improve usability if password self reset is disabled
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Self Service
UCS 4.4
Other Linux
: P5 enhancement (vote)
: UCS 4.4-4-errata
Assigned To: Dirk Wiesenthal
Johannes Keiser
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2020-03-23 22:52 CET by Sönke Schwardt-Krummrich
Modified: 2020-04-22 15:30 CEST (History)
5 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 2: Improvement: Would be a product improvement
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.034
Enterprise Customer affected?:
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments
Screencapture of self service (425.66 KB, image/gif)
2020-03-23 22:52 CET, Sönke Schwardt-Krummrich
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sönke Schwardt-Krummrich univentionstaff 2020-03-23 22:52:39 CET
Created attachment 10331 [details]
Screencapture of self service

If the password reset self service is deactivated via UCR, there is still room for improvement in terms of usability.

The standard login page and the single sign-on login page contain a link ("Forgot password") that refers directly to the self service. In the self service, the three actions/tabs "Forgot password", "Protect account access" and "Change password" are then offered. But only the last item can be used by the user. 
With the first two points, one is first offered a login and then, after entering the credentials, the user is informed that this function has been deactivated via UCR. This warning message is not understood by all users.

Why don't we hide the two actions "Forgot password" and "Protect account access" completely if the UCR variable deactivates this?

See also the attached screencapture.
Comment 1 Ingo Steuwer univentionstaff 2020-03-24 07:28:42 CET
I'ts uncommon to activate the self service without password reset functionality. If needed the links of the SAML login page can be configured to guide the user - see #50609 #50610 #50608 

This Bug should address the links in the self service itself and remove the "password forgotten" functionality if deactivated.
Comment 2 Dirk Wiesenthal univentionstaff 2020-04-20 12:55:31 CEST
Using this bug to make it configurable whether you want to see a subpage (for every subpage):

New:
umc/self-service/passwordchange/frontend/enabled
umc/self-service/passwordreset/frontend/enabled
umc/self-service/protect-account/frontend/enabled

Already existed:
umc/self-service/account-verification/frontend/enabled
umc/self-service/account-registration/frontend/enabled
umc/self-service/profiledata/enabled


Furthermore, we can also disable the backend (raising a UMC Error if the umc action of the self service module is called):

umc/self-service/passwordreset/backend/enabled
umc/self-service/protect-account/backend/enabled

Does not make sense for umc/self-service/passwordchange/backend/enabled (and therefore it does not exist) as the backend functions are not in the UMC module, but are built in the UMC server directly.



If you navigate to the self service page with a certain subpage that is disabled, you now get a 404 message.

univention-self-service 4.0.3-19
univention-management-console 11.0.4-62
Comment 3 Dirk Wiesenthal univentionstaff 2020-04-20 13:42:15 CEST
Added 83_self_service/07_test_frontend_links in
  ucs-test 9.0.3-189A~4.4.0.202004201338

It only tests the frontend UCR variables, though.
Comment 4 Felix Botner univentionstaff 2020-04-21 09:58:04 CEST
please have a look at the test

 83_self_service.01_test_reset_via_email.master091
 83_self_service.04_user_invitation.master091
 83_self_service.07_test_frontend_links.master091

these test are broken now
Comment 5 Dirk Wiesenthal univentionstaff 2020-04-21 18:56:27 CEST
Fixed the tests.
Comment 6 Johannes Keiser univentionstaff 2020-04-22 11:35:56 CEST
OK: 404 page
OK: passwordreset (Password forgotten) - frontend
OK: passwordreset (Password forgotten) - backend
OK: setcontactinformation (Protect account) - frontend
OK: setcontactinformation (Protect account) - backend
OK: passwordchange (Password change) - frontend
OK: passwordchange (Password change) - backend // no UCR var since no umc command
OK: doc
OK: defaults are still the same
OK: yaml


(In reply to Felix Botner from comment #4)
> please have a look at the test
> 
>  83_self_service.01_test_reset_via_email.master091
>  83_self_service.04_user_invitation.master091
>  83_self_service.07_test_frontend_links.master091
> 
> these test are broken now

OK: fixed

-> verified