Task #962: Create PwdResetConfiguration - password-reset - IdStory Identity Manager

Actions

Copy link

Task #962

closed

Create PwdResetConfiguration

Added by Radek Tomiška almost 7 years ago. Updated over 6 years ago.

Status:

Closed

Priority:

Low

Assignee:

Peter Štrunc

Target version:

Start date:

02/13/2018

Due date:

% Done:

100%

Estimated time:

2.00 h

Owner:

Description

Create class PwdResetConfiguration and move all module properties (e.g. verification.ttl, allowed.attrs, passwordReset.public.idm.enabled) into it. See IdentityConfiguration for inspiration.

Related issues

Precedes password-reset - Task #955: Disable password reset in IdM

Closed

Patrik Stloukal

02/14/2018

Actions

Issue # Delay: days Cancel

History
Notes
Property changes

Actions

Copy link

Updated by Radek Tomiška almost 7 years ago

Precedes Task #955: Disable password reset in IdM added

Actions

Copy link

Updated by Patrik Stloukal almost 7 years ago

Status changed from New to Needs feedback
Assignee changed from Patrik Stloukal to Radek Tomiška
% Done changed from 0 to 90

i created interface PwdResetConfiguration and class DefaultPwdResetConfiguration like IdentityConfiguration

commit: https://git.bcvsolutions.eu/modules/password-reset/commit/282253ac798745b89022ce3f9604e21c18430ef4

please look on my implementation and write feedback.

Actions

Copy link

Updated by Radek Tomiška over 6 years ago

Assignee changed from Radek Tomiška to Peter Štrunc

Actions

Copy link

Updated by Peter Štrunc over 6 years ago

Status changed from Needs feedback to In Progress
Assignee changed from Peter Štrunc to Patrik Stloukal

I did review. Why is new configuration not used anywhere?

Actions

Copy link

Updated by Patrik Stloukal over 6 years ago

Assignee changed from Patrik Stloukal to Peter Štrunc

i don't know now, maybe it is not in ticket description or maybe i wanted to be sure it is correctly implemented...

so i use them in code, but property "pwdreset.token.ttl" (as is in documentation) is nowhere used - but property "pwdreset.verification.ttl". I think it is the same property so i replace it..

commit: https://git.bcvsolutions.eu/modules/password-reset/commit/587e121d2f622590bdc54fdf7434f43c84676a45
https://git.bcvsolutions.eu/modules/password-reset/commit/219867f016be308ca127246176f594d8a33e2838

Actions

Copy link