Task #3089
closed
Add synchronization configuration to synchronization log
100%
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.
Updated by Tomáš Doischer over 2 years ago
- Status changed from New to In Progress
Updated by Tomáš Doischer over 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.
Updated by Tomáš Doischer over 2 years ago
- Assignee changed from Tomáš Doischer to Roman Kučera
@kucerar, can you give me feedback please?
Updated by Roman Kučera over 2 years ago
- Status changed from Needs feedback to Resolved
- Assignee changed from Roman Kučera to Tomáš Doischer
Thx, works nicely. Merged
Updated by Tomáš Doischer over 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?
Updated by Roman Kučera over 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