From 15e1a41130281ff32e727e2aafef37cfc7816e5a Mon Sep 17 00:00:00 2001 From: Tim Beale Date: Thu, 9 Aug 2018 15:34:51 +1200 Subject: netcmd: Fix kerberos option for domain backups The previous fix still didn't work if you specified --kerberos=yes (in which case the creds still doesn't have a password). credopts.get_credentials(lp) should be enough to ensure a user/password is set (it's all that the other commands seem to do). BUG: https://bugzilla.samba.org/show_bug.cgi?id=13566 Signed-off-by: Tim Beale Reviewed-by: Andrew Bartlett Reviewed-by: Douglas Bagnall (cherry picked from commit 8fb706c34682bf6dc6033963518c7eccffc3944f) --- python/samba/netcmd/domain_backup.py | 4 ---- 1 file changed, 4 deletions(-) (limited to 'python') diff --git a/python/samba/netcmd/domain_backup.py b/python/samba/netcmd/domain_backup.py index b16763afb15..b2c830259fc 100644 --- a/python/samba/netcmd/domain_backup.py +++ b/python/samba/netcmd/domain_backup.py @@ -152,10 +152,6 @@ def check_targetdir(logger, targetdir): def check_online_backup_args(logger, creds, server, targetdir): # Make sure we have all the required args. - u_p = {'user': creds.get_username(), - 'pass': creds.get_password()} - if None in u_p.values(): - raise CommandError("Creds required.") if server is None: raise CommandError('Server required') -- cgit v1.2.1