Bug 35251 - connector/s4/mapping/group/syncmode=write & group membership
connector/s4/mapping/group/syncmode=write & group membership
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: S4 Connector
UCS 3.2
Other Linux
: P5 normal (vote)
: UCS 3.2-3-errata
Assigned To: Felix Botner
Stefan Gohmann
:
Depends on:
Blocks: 35905
  Show dependency treegraph
 
Reported: 2014-07-03 20:37 CEST by Stefan Gohmann
Modified: 2014-09-11 14:51 CEST (History)
3 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Gohmann univentionstaff 2014-07-03 20:37:27 CEST
Even if connector/s4/mapping/group/syncmode is set to write, group members are synced back from S4 to OpenLDAP when a user was changed. The post hook functions should consider the connector/s4/mapping/group/syncmode variable.
Comment 1 Felix Botner univentionstaff 2014-09-04 13:35:29 CEST
Added sync_mode check to post modify functions object_memberships_sync_from_ucs (return if group sync mode is read) and object_memberships_sync_to_ucs (return if group sync_mode is write).

YAML: 2014-09-03-univention-s4-connector.yaml
Comment 2 Stefan Gohmann univentionstaff 2014-09-09 08:03:24 CEST
Several other sync_mode checks consider the value 'none' too. For example:
  if self.property[property_type].sync_mode in ['write', 'none']:

Could add 'none'?

YAML: OK, small adaption with r53465
Comment 3 Felix Botner univentionstaff 2014-09-09 09:56:08 CEST
(In reply to Stefan Gohmann from comment #2)
> Several other sync_mode checks consider the value 'none' too. For example:
>   if self.property[property_type].sync_mode in ['write', 'none']:
> 
> Could add 'none'?
> 
> YAML: OK, small adaption with r53465

OK
Comment 4 Stefan Gohmann univentionstaff 2014-09-09 10:43:00 CEST
YAML: OK, small adaption with r53465

Code review: OK

UCS 4.0 merge: OK

ucs-test: OK
Comment 5 Janek Walkenhorst univentionstaff 2014-09-10 17:41:11 CEST
http://errata.univention.de/ucs/3.2/199.html