Template workflow allow an empty SSHA password
Template workflow allow an empty SSHA password
There is a problem with the template workflow and/or the handling of password methods, because you can end up with an empty SSHA password doing the following:
- Create a template and fill nothing
- Use this template to create a user
- Fill all asked fields, then just validate
- The user has no password but has SSHA method
Link issues together to show that they're related. Learn more.
Activity
- bmortier changed milestone to %FusionDirectory 1.4
changed milestone to %FusionDirectory 1.4
By Côme Chilliet on 2019-10-08T14:43:54 (imported from GitLab)
- bmortier added PJ1811-0242 label
added PJ1811-0242 label
- bmortier added fusiondirectory-core label
added fusiondirectory-core label
- bmortier changed due date to October 15, 2019
changed due date to October 15, 2019
By bmortier on 2019-10-09T09:04:28 (imported from GitLab)
- bmortier mentioned in issue #6034
mentioned in issue #6034
By Côme Chilliet on 2019-10-10T06:57:56 (imported from GitLab)
- bmortier created merge request !690 to address this issue
created merge request !690 to address this issue
By Côme Chilliet on 2019-10-10T09:15:08 (imported from GitLab)
- bmortier mentioned in merge request !690
mentioned in merge request !690
By Côme Chilliet on 2019-10-10T09:15:08 (imported from GitLab)
So it appears there is a problem with #5834 (closed)
%askme%
is only available in SelectAttribute when editing a template, not when creating one.This needs to be fixed first and then we should be able to set
%askme%
in userPassword by default to avoid problems.By Côme Chilliet on 2019-10-10T09:42:01 (imported from GitLab)
Edited by bmortier- bmortier added 1h of time spent at 2019-10-10
added 1h of time spent at 2019-10-10
By Côme Chilliet on 2019-10-10T09:41:36 (imported from GitLab)
- bmortier mentioned in issue #6039
mentioned in issue #6039
By Côme Chilliet on 2019-10-15T08:21:04 (imported from GitLab)
- bmortier created merge request !697 to address this issue
created merge request !697 to address this issue
By Côme Chilliet on 2019-10-16T13:10:21 (imported from GitLab)
- bmortier mentioned in merge request !697
mentioned in merge request !697
By Côme Chilliet on 2019-10-16T13:10:21 (imported from GitLab)
So now
%askme%
value is in userPassword by default. This is better because it means applying a template with all fields untouched to a user will not change its password, and creating a user will ask for a password, which both makes sense.By Côme Chilliet on 2019-10-16T13:20:18 (imported from GitLab)
- bmortier added 20m of time spent at 2019-10-16
added 20m of time spent at 2019-10-16
By Côme Chilliet on 2019-10-16T13:20:18 (imported from GitLab)
- bmortier added To Be Tested label
added To Be Tested label
- bmortier assigned to @jswaelens and unassigned @MCMic
assigned to @jswaelens and unassigned @MCMic
By Côme Chilliet on 2019-10-16T13:20:20 (imported from GitLab)
- bmortier closed
closed
By Jonathan Swaelens on 2019-10-22T10:08:47 (imported from GitLab)
- bmortier added 5m of time spent at 2019-10-22
added 5m of time spent at 2019-10-22
By Jonathan Swaelens on 2019-10-22T10:08:48 (imported from GitLab)
- bmortier removed due date
removed due date
By Jonathan Swaelens on 2019-10-22T10:08:48 (imported from GitLab)
- bmortier added Fixed label and removed To Be Tested label
added Fixed label and removed To Be Tested label
- bmortier marked this issue as related to #6039
marked this issue as related to #6039
By bmortier on 2021-01-28T20:53:36 (imported from GitLab)
- bmortier marked this issue as related to #6034
marked this issue as related to #6034
By bmortier on 2021-01-28T20:56:16 (imported from GitLab)
- bmortier changed the description
changed the description
By bmortier on 2021-01-28T20:56:22 (imported from GitLab)