Bug 47494 - join scripts do not receive LDAP credentials when run on DC master
join scripts do not receive LDAP credentials when run on DC master
Status: RESOLVED WONTFIX
Product: UCS
Classification: Unclassified
Component: Join (univention-join)
UCS 4.3
Other Linux
: P5 normal (vote)
: ---
Assigned To: UCS maintainers
UCS maintainers
:
: 52131 (view as bug list)
Depends on: 47497
Blocks:
  Show dependency treegraph
 
Reported: 2018-08-08 10:38 CEST by Daniel Tröder
Modified: 2021-05-14 16:38 CEST (History)
2 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 2: Will only affect a 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.069
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

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Tröder univentionstaff 2018-08-08 10:38:14 CEST
When a join script runs, LDAP admin credentials are supplied as "--binddn $DN" and "--bindpwd $PW" and are passed to programs that require them through "$@" - except when a join script is run on a DC master.

Thus programs that get passed "$@" cannot rely on it, but must retrieve LDAP credentials on their own, scattering open(ldap.secret).read() all over the place.

(Including in ISV code, that we cannot fix, once we move to a more secure password location.)

The LDAP admin credentials in "$@" is a undocumented feature. There are references to it in the developer documentation that "$@" should be passed to UDM calls, but non explain what's in it and where it comes from. And especially not how to cope with the missing credentials in case of DC master.

1) The API for join scripts should be equal on all server roles. Add "--binddn $DN" and "--bindpwd $PW" to "$@" of join scripts run on a DC master.

2) Document this feature in the developer documentation. Not making a separate bug for this, because documenting unexpected behavior can IMHO also be considered a fix.
(So 1. is optional, but 2. is mandatory.)
Comment 1 Felix Botner univentionstaff 2018-08-08 10:55:20 CEST
please do no longer use or document --bindpwd, instead use --bindpwdfile, see Bug #46842 for more information
Comment 2 Daniel Tröder univentionstaff 2018-08-08 11:18:39 CEST
I have created a separate bug to document the new join script API: Bug #47497.

Whether it is active on a DC master or not, must still be decided and documented in this bug.
Comment 3 Philipp Hahn univentionstaff 2020-10-08 19:03:10 CEST
*** Bug 52131 has been marked as a duplicate of this bug. ***
Comment 4 Ingo Steuwer univentionstaff 2021-05-14 16:38:23 CEST
This issue has been filed against UCS 4.3.

UCS 4.3 is out of maintenance and many UCS components have 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 it and update the UCS version. In this case please provide detailed information on how this issue is affecting you.