Project

General

Profile

Feature #2212

Move MultipleSecureTextFormPropertyConverter from Extras module to product.

Added by Roman Kučera 5 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Connectors
Target version:
Start date:
04/16/2020
Due date:
% Done:

100%

Estimated time:

Description

In Extras module there is new converter for connid connectors. This converter is mandatory for WinRM+AD connector version 1.0.3 or newer.
The point of this ticket is to move this converter to product so we can use clean product with connector without extras module.


Related issues

Related to CzechIdM - Task #2251: PropertyConvertor: add unit test for property convertorsNew05/13/2020

History

#1 Updated by Roman Kučera 5 months ago

  • Assignee set to Roman Kučera

#2 Updated by Roman Kučera 5 months ago

  • Status changed from New to Needs feedback
  • Assignee changed from Roman Kučera to Vít Švanda
  • % Done changed from 0 to 90

Implementation in https://github.com/bcvsolutions/CzechIdMng/commit/3d5e19d9039237fa15f45617aa66c3d680c91f40
So now if some connector will use GuardedString array in configuration, IdM will support it. To this point it was not supported.
This was developed for Extras module 1.8.0 but it will be nice to have it in product.

Pull request was also created. Vitek could you please make a review?

#3 Updated by Vít Švanda 5 months ago

  • Assignee changed from Vít Švanda to Radek Tomiška

#4 Updated by Radek Tomiška 4 months ago

  • Category set to Connectors
  • Status changed from Needs feedback to Resolved
  • Assignee changed from Radek Tomiška to Roman Kučera
  • % Done changed from 90 to 100

I did test and code review, it works, thx!

Note: test is missing, but all supported convertors should be tested as well - #2251.

#5 Updated by Radek Tomiška 4 months ago

  • Related to Task #2251: PropertyConvertor: add unit test for property convertors added

#6 Updated by Radek Tomiška 4 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF

Go to top