Project

General

Profile

Task #1405

Modify Workflow sync Ldap groups

Added by Patrik Stloukal about 1 month ago. Updated about 15 hours ago.

Status:
In Progress
Priority:
Normal
Category:
Workflow
Target version:
Start date:
12/07/2018
Due date:
% Done:

90%


Description

In workflow change default validity of roles assigned based on membership in LDAP to null and assigne this membership to all valid contracts or valid in future.


Related issues

Related to CzechIdM - Task #1268: Improve the AD group synchronization workflow Closed 09/24/2018

History

#1 Updated by Vít Švanda about 1 month ago

May be it is good time for move that workflow from product to the Extras module.

#2 Updated by Marcel Poul about 1 month ago

  • Related to Task #1268: Improve the AD group synchronization workflow added

#3 Updated by Ondřej Kopr about 1 month ago

  • Target version set to Onyx (9.3.2)

#4 Updated by Peter Šourek 29 days ago

I saw it working. Issues found were fixed. Please add possibility to create all roles in one defined catalogue.

#5 Updated by Ondřej Kopr 29 days ago

  • Target version changed from Onyx (9.3.2) to Opal (9.4.0)

#6 Updated by Patrik Stloukal 10 days ago

  • Status changed from New to In Progress
  • Assignee changed from Patrik Stloukal to Vít Švanda
  • % Done changed from 0 to 90

I added possibility to create all roles in one defined catalogue, modified documentation (https://wiki.czechidm.com/tutorial/dev/ad_groups_sync_workflow?s[]=workflow&s[]=groups#set_aplication_properties), and i tested this new feature on ours AD system.

commits: https://github.com/bcvsolutions/CzechIdMng/commit/5bd39341e6dee36daae146dce9c8626da9ed2945
https://github.com/bcvsolutions/CzechIdMng/commit/fc2397f8b74838a41921109655e421d233363a46

Vít please look on this new version, it should be ok

#7 Updated by Vít Švanda 9 days ago

  • Status changed from In Progress to Needs feedback

#8 Updated by Vít Švanda 2 days ago

  • Target version changed from Opal (9.4.0) to Opal (9.4.0-rc.2)

#9 Updated by Vít Švanda about 15 hours ago

  • Status changed from Needs feedback to In Progress
  • Assignee changed from Vít Švanda to Patrik Stloukal

I did code review.

- For the next time, please avodi using the fix method order in tests (@FixMethodOrder + methods with prefix n1... is not nice).
- For the next please create the pull request.
- In code of WF are some places from whitch I am not happy:

  • Why are variables named as constants (CREATE_ALL_ROLES_TO_ONE_CATALOGUE....),
  • Configuration keys, should be in configuration objects (as getters).
  • :-) logItem.addToLog("loooooool " + attr.getName()); ??
  • For joining a strings we using MessageFormat not "+".

Also available in: Atom PDF