Project

General

Profile

Task #813

Identity configuration - make contract properties private, add default contract state

Added by Radek Tomiška almost 4 years ago. Updated 29 days ago.

Status:
Closed
Priority:
Normal
Category:
Configuration
Target version:
Start date:
11/02/2017
Due date:
% Done:

100%

Estimated time:
Milestones:

Description

Identity configuration is whole public now (https://wiki.czechidm.com/devel/dev/configuration/backend#identity). Property:
  • idm.pub.core.identity.create.defaultContract.enabled
    should be private, because she is not needed on FE. Other properties are needed on FE to show / hide some agendas and have to be public.
New private properties:
  • idm.sec.core.identity.create.defaultContract.state
  • idm.sec.core.identity.create.defaultContract.expiration
    will be added - initial default contract state is needed to higher flexibility. In some project is useful to create default contract in EXCLUDED or DISABLED state. Expriration in days will be used for setting default contract's validTilll attribute.
Implementation notes:
  • IdentityConfiguration should be splited - IdentityConfiguration (~ public) and PrivateIdentityConfiguration
  • PrivateIdentityConfiguration will provide IdentityConfiguration (see ProvisioningConfiguration - ProvisioningBreakConfiguration)
  • PrivateIdentityConfigurationwill contain three properties above about defaultContract

Change log is needed - changing property key => product implementers should be aware about it.
Change script will be provided - changing property key => change affects only configuration in property files. Configuration in db will be updated automatically.


Related issues

Related to CzechIdM - Task #1212: New identity is created with state 'CREATED' New08/14/2018

Related to CzechIdM - Feature #1028: 1) upload and display image / delete imageClosed03/13/2018

History

#1 Updated by Radek Tomiška almost 4 years ago

  • Description updated (diff)

#2 Updated by Radek Tomiška almost 4 years ago

  • Description updated (diff)

#3 Updated by Radek Tomiška over 2 years ago

  • Related to Task #1212: New identity is created with state 'CREATED' added

#4 Updated by Radek Tomiška over 2 years ago

  • Subject changed from Identity configuration - make contract properties private to Identity configuration - make contract properties private, add default contract state

#5 Updated by Radek Tomiška 3 months ago

  • Description updated (diff)
  • Target version set to 11.2.0

#6 Updated by Radek Tomiška 3 months ago

  • Related to Feature #1028: 1) upload and display image / delete image added

#7 Updated by Radek Tomiška 3 months ago

  • Status changed from New to In Progress

#8 Updated by Radek Tomiška 3 months ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Radek Tomiška to Ondrej Husník
  • % Done changed from 0 to 90

Feature is implemented and is backward compatible. Previous configuration property is deprecated only and is used, if new property is not configured.

Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/d16d3c0c64c055fc31cb366bc6d57b6bee1a8b53

Doc:
https://wiki.czechidm.com/devel/documentation/application_configuration/dev/backend#identity

Could you provide me a feedback, please?

#9 Updated by Radek Tomiška about 1 month ago

I improved loading of configured contract state to be more user friendly - default state is returned, when configuration is wrong + warn log (the same behavior as in other configuration).
I added user friendly validation, when contract state configuration is saved from UI.

Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/b5bb8c3c648ad5e838e69e441a1c70d0da13228d

#10 Updated by Ondrej Husník about 1 month ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Ondrej Husník to Radek Tomiška
  • % Done changed from 90 to 100

The properties work fine. Thank you for checking state property a wrong value of which caused confusing situation in IdM in my opinion.

#11 Updated by Radek Tomiška 29 days ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF

Go to top