Project

General

Profile

Actions
Copy link

Task #3089

closed

Add synchronization configuration to synchronization log

Added by Tomáš Doischer about 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Tomáš Doischer
Category:
Synchronization
Target version:
12.2.0
Start date:
03/28/2022
Due date:
% Done:

100%

Estimated time:
Owner:

Description

Currently, only the "specific configuration" is saved to the synchronization log. We would also like to have the information about tokens (start and end tokens) and configured synchronization action types.

Actions
Copy link
 #1

Updated by Tomáš Doischer about 2 years ago

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

Updated by Tomáš Doischer about 2 years ago

  • Status changed from In Progress to Needs feedback
  • % Done changed from 0 to 100

Implemented. I added:

  • mapping and correlation attribute name (this is embedded in config anyway)
  • whether the synchronization is a reconciliation
  • the initial token info (the end token is already present)
  • the synchronization action types and WF keys (those are not localized but still should be useful)

No extra DB query is needed and the details are saved to the log only once so there will be no performance hit.

PR: https://github.com/bcvsolutions/CzechIdMng/pull/187

Actions
Copy link
 #3

Updated by Tomáš Doischer about 2 years ago

  • Assignee changed from Tomáš Doischer to Roman Kučera

@kucerar, can you give me feedback please?

Actions
Copy link
 #4

Updated by Roman Kučera about 2 years ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Roman Kučera to Tomáš Doischer

Thx, works nicely. Merged

Actions
Copy link
 #5

Updated by Tomáš Doischer almost 2 years ago

  • Status changed from Resolved to In Progress
  • Assignee changed from Tomáš Doischer to Roman Kučera
  • % Done changed from 100 to 90

One issue was found during further testing, tree node and role catalogue synchronizations overload the process() method which creates the log. As a result, this feature didn't work correctly. I made a necessary change and tested it, it works well.

PR: https://github.com/bcvsolutions/CzechIdMng/pull/210

@kucerar, can you take a look at it, please?

Actions
Copy link
 #6

Updated by Roman Kučera almost 2 years ago

  • Status changed from In Progress to Resolved
  • Assignee changed from Roman Kučera to Tomáš Doischer
  • % Done changed from 90 to 100

LGTM, thanks, merged

Actions
Copy link
 #7

Updated by Roman Kučera almost 2 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF