Bug 32324 - System setup fails with long or Umlaut DN user
System setup fails with long or Umlaut DN user
Status: RESOLVED WONTFIX
Product: UCS
Classification: Unclassified
Component: System setup
UCS 3.2
Other Linux
: P5 normal (vote)
: UCS 3.x
Assigned To: UCS maintainers
:
Depends on: 16210
Blocks: 28670 32005 33091
  Show dependency treegraph
 
Reported: 2013-08-23 08:16 CEST by Philipp Hahn
Modified: 2018-04-14 13:37 CEST (History)
4 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): Cleanup, Internationalization, UCS Performance
Max CVSS v3 score:
hahn: Patch_Available+


Attachments
Fix base/univention-system-setup/usr/lib/univention-system-setup/scripts/setup_utils.sh (33.66 KB, patch)
2013-08-23 09:08 CEST, Philipp Hahn
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2013-08-23 08:16:07 CEST
+++ This bug was initially created as a clone of Bug #16210 +++
univention-syste-setup fails for users with long DNs or Umlauts in their DN:
ldapsearch-wrapper and ldapseach-base64 are missing:

@ base/univention-system-setup/usr/lib/univention-system
ldap_binddn ()
-setup/scripts/setup_utils.sh
-                       dn=`ldapsearch -x -ZZ -D "$ldap_hostdn" -y /etc/machine.secret -h $ldap_master "(&(objectClass=person)(uid=$ldap_username))" | grep "dn: " | sed -e 's|dn: ||' | head -n 1`
-                       echo "$dn"
+                       ldapsearch -x -ZZ -D "$ldap_hostdn" -y /etc/machine.secret -h "$ldap_master" "(&(objectClass=person)(uid=$ldap_username))" dn |
+                       ldapsearch-wrapper |
+                       ldapsearch-base64 |
+                       sed -ne 's|^dn: ||;T;p;q'
Comment 1 Philipp Hahn univentionstaff 2013-08-23 09:07:46 CEST
e4146e1 Bug #32324: USS: Fix local variables
f21f84d Bug #32324: USS: Fix ldapsearch wrapping and encoding
bee100f Bug #32324: USS: Fix shell variable quoting
d612a5b Bug #32324: USS: Use invoke-rc.d
d15caa8 Bug #32324: USS: Fix comments
7a7c0e4 Bug #32324: USS: add regression test
Comment 2 Philipp Hahn univentionstaff 2013-08-23 09:08:20 CEST
Created attachment 5391 [details]
Fix base/univention-system-setup/usr/lib/univention-system-setup/scripts/setup_utils.sh
Comment 3 Stefan Gohmann univentionstaff 2013-08-23 09:21:23 CEST
(In reply to Philipp Hahn from comment #1)
> e4146e1 Bug #32324: USS: Fix local variables
> f21f84d Bug #32324: USS: Fix ldapsearch wrapping and encoding
> bee100f Bug #32324: USS: Fix shell variable quoting
> d612a5b Bug #32324: USS: Use invoke-rc.d
> d15caa8 Bug #32324: USS: Fix comments
> 7a7c0e4 Bug #32324: USS: add regression test

OK nice. I think it would be better to split the bug in one cleanup bug and one bug for the long DN issue. That makes it easier to calculate the effect.
Comment 4 Philipp Hahn univentionstaff 2013-09-05 16:20:41 CEST
Part of that patch was applied for Bug #32463.
Comment 5 Stefan Gohmann univentionstaff 2017-06-16 20:38:34 CEST
This issue has been filed against UCS 3. UCS 3 is out of the normal maintenance and many UCS components have vastly changed in UCS 4.

If this issue is still valid, please change the version to a newer UCS version otherwise this issue will be automatically closed in the next weeks.
Comment 6 Stefan Gohmann univentionstaff 2017-08-08 07:11:44 CEST
This issue has been filed against UCS 3.2.

UCS 3.2 is out of maintenance and many UCS components have vastly changed in later releases. Thus, this issue is now being closed.

If this issue still occurs in newer UCS versions, please use "Clone this bug" or reopen this issue. In this case please provide detailed information on how this issue is affecting you.