Task #767
closed
Password change - include additional attributes
100%
Description
When password is changed, then only PASSWORD attribute is provisioned now. We need to do provisioning for additional attributes, when password is changed (e.g. password expiration).
New features will be added:
- flag to attribute mapping - "include, if password is changed"
- attributes wuth this flag will be send with new password, when password is changed
- two ways for sending additional attributes will be implemented:
-- send additional attributes together with new password in one provisioning operation
-- send additional attributes after password is successfully changed in another provisioning operation
-- two ways will be configurable by application configuration
Updated by Radek Tomiška over 6 years ago
Every connector can implement update method and password change differently:
https://github.com/Tirasa/ConnIdADBundle/blob/master/src/main/java/net/tirasa/connid/bundles/ad/ADConnector.java#L181
https://github.com/Tirasa/ConnIdDBBundle/blob/master/table/src/main/java/net/tirasa/connid/bundles/db/table/DatabaseTableConnector.java#L475
This is reason why two ways for sending additional attributes will be implemented:
- e.g. AD - two provisioning operations
- e.g. Table - one provisioning operation
Updated by Radek Tomiška over 6 years ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Radek Tomiška to Ondřej Kopr
- % Done changed from 0 to 90
It's implemented as description says. Tests and documentation is included.
Documentation:
https://wiki.czechidm.com/devel/dev/system/system-mapping#send_additional_attributes_with_password
https://github.com/bcvsolutions/CzechIdMng/blob/develop/CHANGELOG.md#provisioning
https://wiki.czechidm.com/devel/dev/configuration/backend#provisioning
Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/5c95ac058656f4c34e9adcbc2ffd34dbf79ac3d1
Could you pls do a review?
Updated by Vít Švanda over 6 years ago
- Assignee changed from Ondřej Kopr to Vít Švanda
Updated by Vít Švanda over 6 years ago
I did review and test all causes (I hope). Everything works fine. Thanks for that.
Updated by Vít Švanda over 6 years ago
- Status changed from Needs feedback to Closed
- Assignee changed from Vít Švanda to Radek Tomiška
- % Done changed from 90 to 100