Bug 35251 – connector/s4/mapping/group/syncmode=write & group membership

Bug 35251 - connector/s4/mapping/group/syncmode=write & group membership


Summary:	connector/s4/mapping/group/syncmode=write & group membership

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	S4 Connector
Version:	UCS 3.2
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 3.2-3-errata
Assigned To:	Felix Botner
QA Contact:	Stefan Gohmann

URL:
Keywords:

Depends on:
Blocks:	35905
	Show dependency tree / graph

Reported:	2014-07-03 20:37 CEST by Stefan Gohmann
Modified:	2014-09-11 14:51 CEST (History)
CC List:	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:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Stefan Gohmann

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

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

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

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

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

2014-09-10 17:41:11 CEST

http://errata.univention.de/ucs/3.2/199.html