Bug 55582 - Server password change breaks AD membermode in univention-samba is uninstalled
Server password change breaks AD membermode in univention-samba is uninstalled
Status: RESOLVED DUPLICATE of bug 54390
Product: UCS
Classification: Unclassified
Component: Samba
UCS 5.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: Samba maintainers
Samba maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2023-01-19 12:51 CET by Arvid Requate
Modified: 2023-03-03 15:43 CET (History)
8 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?: 2: Will only affect a few installed domains
How will those affected feel about the bug?: 5: Blocking further progress on the daily work
User Pain: 0.286
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2022122121000198
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 Arvid Requate univentionstaff 2023-01-19 12:51:24 CET
In a customer environment with AD membermode configuration the package univention-samba was uninstalled (maybe because the app Agorum wants to claim the SMB ports) and that will break server password change in a subtle way, as the code that actually changes the password in AD is currently located in:

services/univention-samba/server_password_change.d/univention-samba

The central lines of code are:
==================
        # change password on ad in member mode
        if samba_role == 'memberserver' and univention.lib.admember.is_localhost_in_admember_mode(ucr=ucr):
                cmd = ['/usr/bin/net', 'ads', 'password', '-P']
                cmd.append('%s$' % ucr.get('hostname', '').upper())
                cmd.append(machine_password)
                process = subprocess.Popen(cmd)
                process.wait()
==================

As discussed with Support and Julia, we need to move the admember specific parts of that code to a new location. I propose this location:

services/univention-ad-conector/server_password_change.d/univention-admember
Comment 1 Felix Botner univentionstaff 2023-03-03 15:43:36 CET

*** This bug has been marked as a duplicate of bug 54390 ***