Bug 45221

Summary: wrong regex for sed to find binddn in univention-run-join-scripts
Product: UCS Reporter: Nico Stöckigt <stoeckigt>
Component: Join (univention-join)Assignee: Jürn Brodersen <brodersen>
Status: CLOSED DUPLICATE QA Contact: Richard Ulmer <ulmer>
Severity: normal    
Priority: P5 CC: best, brodersen, gohmann
Version: UCS 4.2Flags: best: Patch_Available+
Target Milestone: UCS 4.3-0-errata   
Hardware: Other   
OS: Linux   
What kind of report is it?: Bug Report What type of bug is this?: 4: Minor Usability: Impairs usability in secondary scenarios
Who will be affected by this bug?: 1: Will affect a very few installed domains How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.046 Enterprise Customer affected?: Yes
School Customer affected?: ISV affected?:
Waiting Support: Flags outvoted (downgraded) after PO Review:
Ticket number: 2017081421000228, 2018011721000475 Bug group (optional):
Max CVSS v3 score:
Bug Depends on:    
Bug Blocks: 47102, 47103    
Attachments: univention-run-join-scripts.patch
patch

Description Nico Stöckigt univentionstaff 2017-08-18 18:07:12 CEST
Created attachment 9125 [details]
univention-run-join-scripts.patch

The attached patch fixed line 159 where the sed regex was wrong - the following sed calls are correct.

The behavior was only wrong in certain environments, when there is a schema extension which affects the result. In this particular case the schema extension deputy (Stellvertreter) is installed and the uid=Administrator is registered as deputy, so it's DN occurs multiple times. Only the Object DN itself starts at the beginning of the line so the regex needs to be like 's/^DN: //'
Comment 1 Florian Best univentionstaff 2017-08-18 18:17:16 CEST
Yes! It can be reproduced with:

udm users/user modify --dn "uid=Administrator,cn=users,$(ucr get ldap/base)" --set description="DN: foobar!"

→ Then binddn contains something like "uid=Administrator,cn=users,dc=school,dc=local\n  description: foobar!"
Comment 2 Florian Best univentionstaff 2017-09-01 12:45:09 CEST
Created attachment 9165 [details]
patch

And some more fixes.
Comment 3 Stefan Gohmann univentionstaff 2018-03-15 06:31:05 CET
Move to 4.3-0-errata. If a UCS 4.2 backport is needed, please clone this issue.
Comment 4 Jürn Brodersen univentionstaff 2018-05-29 16:49:40 CEST
(In reply to Florian Best from comment #2)
> Created attachment 9165 [details]
> patch
> 
> And some more fixes.

Moved to bug 47102 and bug 47103.
Comment 5 Jürn Brodersen univentionstaff 2018-05-29 16:50:12 CEST

*** This bug has been marked as a duplicate of bug 36097 ***
Comment 6 Stefan Gohmann univentionstaff 2018-06-27 08:25:10 CEST
Nothing to release.