Project

General

Profile

Actions

Defect #2708

closed

AD Wizard for users has to contain valid, existing DN for disabled accounts even if account protection not used

Added by Ondrej Husník almost 4 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Low
Assignee:
Ondrej Husník
Category:
Wizard
Target version:
Start date:
03/04/2021
Due date:
% Done:

100%

Estimated time:
4.00 h
Affected versions:
Owner:

Description

Add relation between Activate protected mode switch and FE validation (mandatory field) of the corresponding DN editbox so that if Protected Mode not enabled ignore content of the DN editbox.


Files


Related issues

Related to IdStory Identity Manager - Task #2695: Testing and release of version 10.8.0ClosedOndrej Husník03/01/2021

Actions
Actions #1

Updated by Radek Tomiška almost 4 years ago

  • Related to Task #2695: Testing and release of version 10.8.0 added
Actions #2

Updated by Vít Švanda almost 4 years ago

  • Category changed from UX to Wizard
  • Target version set to 11.0.0
Actions #3

Updated by Vít Švanda almost 4 years ago

  • Estimated time set to 4.00 h
Actions #4

Updated by Vít Švanda over 3 years ago

  • Tracker changed from Task to Defect
  • Affected versions 10.8.0, 10.8.1, 10.8.2 added
Actions #5

Updated by Ondrej Husník over 3 years ago

  • Status changed from New to In Progress
Actions #6

Updated by Vít Švanda over 3 years ago

I modified how wizard works with delete user container on BE (is persist only if protected mode is active now).

Commit: https://github.com/bcvsolutions/CzechIdMng/commit/1a3c0e2ab8ef1fb346a6a7eaf61c142125094d8b

Actions #7

Updated by Ondrej Husník over 3 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Ondrej Husník to Vít Švanda

Currently if account protected mode is not used, the value of disabledUsers container value is nulled and the test existence of the container on the target AD makes no sense and is skipped.

Please provide me a feedback.
https://github.com/bcvsolutions/CzechIdMng/commit/5bed7e48d64138e91a6c5b26bac2d0f10ac824d2

Actions #8

Updated by Ondrej Husník over 3 years ago

  • % Done changed from 0 to 90
Actions #9

Updated by Vít Švanda over 3 years ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Vít Švanda to Ondrej Husník
  • % Done changed from 90 to 100

I did review and tested it. Works fine now. Thanks for that.

Actions #10

Updated by Ondrej Husník over 3 years ago

  • Status changed from Resolved to In Progress
  • % Done changed from 100 to 80

While using IdM I found a regression related to the changes allowing null value in deletedUsersContainer, performed in this modification:
Vít Švanda wrote:

I modified how wizard works with delete user container on BE (is persist only if protected mode is active now).

Commit: https://github.com/bcvsolutions/CzechIdMng/commit/1a3c0e2ab8ef1fb346a6a7eaf61c142125094d8b

The cause of problem is in getDefaultDN script line 51. there has to be treated null value of the deletedOu variable. Something like this
if (deletedOu != null && originalDn.endsWith(deletedOu)) {...

Without this fix there works creating of a new account on AD but update doesn't. Will be solved in #2739

Actions #11

Updated by Ondrej Husník over 3 years ago

  • % Done changed from 80 to 100
Actions #12

Updated by Ondrej Husník over 3 years ago

  • Status changed from In Progress to Resolved
Actions #13

Updated by Radek Tomiška over 3 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF