Project

General

Profile

Actions
Copy link

Task #1829

closed

Disable/enable password generating trough IdM

Added by Ondřej Kopr over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Peter Štrunc
Target version:
1.6.0
Start date:
09/02/2019
Due date:
% Done:

100%

Estimated time:
Owner:

Description

Please allow disabling password generating trough CzechIdM.

You can use same configuration property as for password reset:

idm.pub.pwdreset.identity.passwordReset.public.idm.enabled
Actions
Copy link
 #1

Updated by Ondřej Kopr over 5 years ago

  • Status changed from New to In Progress
Actions
Copy link
 #3

Updated by Ondřej Kopr over 5 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Ondřej Kopr to Peter Štrunc
  • % Done changed from 0 to 90

Thank you. Now this feature works also for generation. Please Peter could you make a review?

Commit (test are included): https://git.bcvsolutions.eu/modules/password-reset/commit/cda5fa26993fc389d15e97689f16b7730c369fd2 (branch: develop)
Updated documentation: https://wiki.czechidm.com/tutorial/adm/modules_pwdreset#configuration

Actions
Copy link
 #4

Updated by Peter Štrunc over 5 years ago

Both code and documentation looks OK. No sonar issues were introduced and generating of password works as intended.

Issues:
  • Property is not working on normal password reset (using email token). I set the property to false and when i finish password reset i am not automatically logged in (which is correct). But password is reset in CzechIdM.
Actions
Copy link
 #5

Updated by Ondřej Kopr over 5 years ago

Peter Šourek wrote:

Issues:
  • Property is not working on normal password reset (using email token). I set the property to false and when i finish password reset i am not automatically logged in (which is correct). But password is reset in CzechIdM.

This is issue of ticket #955 in this ticket was made changes that broke this feature, configuration property was changed from idm.pub.pwdreset.identity.passwordReset.public.idm.enabled to pwdResetConfiguration.isPwdResetIdmEnabled(). I already created new ticket for fix #1830

Can i close this ticket?

Actions
Copy link
 #6

Updated by Ondřej Kopr over 5 years ago

  • Status changed from Needs feedback to Closed
  • % Done changed from 90 to 100
Actions
Copy link

Also available in: Atom PDF