Bug 45221 - wrong regex for sed to find binddn in univention-run-join-scripts
wrong regex for sed to find binddn in univention-run-join-scripts
Status: CLOSED DUPLICATE of bug 36097
Product: UCS
Classification: Unclassified
Component: Join (univention-join)
UCS 4.2
Other Linux
: P5 normal (vote)
: UCS 4.3-0-errata
Assigned To: Jürn Brodersen
Richard Ulmer
:
Depends on:
Blocks: 47102 47103
  Show dependency treegraph
 
Reported: 2017-08-18 18:07 CEST by Nico Stöckigt
Modified: 2018-06-27 08:25 CEST (History)
3 users (show)

See Also:
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:
best: Patch_Available+


Attachments
univention-run-join-scripts.patch (575 bytes, patch)
2017-08-18 18:07 CEST, Nico Stöckigt
Details | Diff
patch (5.23 KB, patch)
2017-09-01 12:45 CEST, Florian Best
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.