Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package yast2-auth-client for 
openSUSE:Factory checked in at 2021-05-08 22:07:11
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-auth-client (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-auth-client.new.2988 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-auth-client"

Sat May  8 22:07:11 2021 rev:43 rq:890928 version:4.4.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-auth-client/yast2-auth-client.changes      
2021-05-02 18:36:05.736953030 +0200
+++ 
/work/SRC/openSUSE:Factory/.yast2-auth-client.new.2988/yast2-auth-client.changes
    2021-05-08 22:07:13.385793122 +0200
@@ -1,0 +2,8 @@
+Wed May  5 10:21:50 UTC 2021 - Samuel Cabrero <scabr...@suse.de>
+
+- Fix an error importing the configuration when the json does not
+  contain all keys; (bsc#1184630);
+- Fix nss_ldap package unnecessarily installed; (bsc#1185499);
+- 4.4.1
+
+-------------------------------------------------------------------

Old:
----
  yast2-auth-client-4.4.0.tar.bz2

New:
----
  yast2-auth-client-4.4.1.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-auth-client.spec ++++++
--- /var/tmp/diff_new_pack.xSiCFq/_old  2021-05-08 22:07:13.853791095 +0200
+++ /var/tmp/diff_new_pack.xSiCFq/_new  2021-05-08 22:07:13.853791095 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-auth-client
-Version:        4.4.0
+Version:        4.4.1
 Release:        0
 URL:            https://github.com/yast/yast-auth-client
 Summary:        YaST2 - Centralised System Authentication Configuration

++++++ yast2-auth-client-4.4.0.tar.bz2 -> yast2-auth-client-4.4.1.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-auth-client-4.4.0/package/yast2-auth-client.changes 
new/yast2-auth-client-4.4.1/package/yast2-auth-client.changes
--- old/yast2-auth-client-4.4.0/package/yast2-auth-client.changes       
2021-04-30 18:13:18.000000000 +0200
+++ new/yast2-auth-client-4.4.1/package/yast2-auth-client.changes       
2021-05-06 11:13:24.000000000 +0200
@@ -1,4 +1,12 @@
 -------------------------------------------------------------------
+Wed May  5 10:21:50 UTC 2021 - Samuel Cabrero <scabr...@suse.de>
+
+- Fix an error importing the configuration when the json does not
+  contain all keys; (bsc#1184630);
+- Fix nss_ldap package unnecessarily installed; (bsc#1185499);
+- 4.4.1
+
+-------------------------------------------------------------------
 Tue Apr 20 13:51:55 UTC 2021 - Ladislav Slez??k <lsle...@suse.cz>
 
 - 4.4.0 (bsc#1185510)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-auth-client-4.4.0/package/yast2-auth-client.spec 
new/yast2-auth-client-4.4.1/package/yast2-auth-client.spec
--- old/yast2-auth-client-4.4.0/package/yast2-auth-client.spec  2021-04-30 
18:13:18.000000000 +0200
+++ new/yast2-auth-client-4.4.1/package/yast2-auth-client.spec  2021-05-06 
11:13:24.000000000 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-auth-client
-Version:        4.4.0
+Version:        4.4.1
 Release:        0
 Url:            https://github.com/yast/yast-auth-client
 Summary:        YaST2 - Centralised System Authentication Configuration
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-auth-client-4.4.0/src/lib/auth/authconf.rb 
new/yast2-auth-client-4.4.1/src/lib/auth/authconf.rb
--- old/yast2-auth-client-4.4.0/src/lib/auth/authconf.rb        2021-04-30 
18:13:18.000000000 +0200
+++ new/yast2-auth-client-4.4.1/src/lib/auth/authconf.rb        2021-05-06 
11:13:24.000000000 +0200
@@ -475,12 +475,18 @@
 
         # Set configuration for LDAP from exported objects.
         def ldap_import(exported_conf)
-            @ldap_conf = exported_conf['conf']
-            @ldap_conf = {} if @ldap_conf.nil?
-            @ldap_pam = exported_conf['pam']
-            @ldap_pam = false if @ldap_pam.nil?
-            @ldap_nss = exported_conf['nss']
-            @ldap_nss = [] if @ldap_nss.nil?
+            if exported_conf.nil?
+                @ldap_conf = {}
+                @ldap_pam = false
+                @ldap_nss = []
+            else
+                @ldap_conf = exported_conf['conf']
+                @ldap_conf = {} if @ldap_conf.nil?
+                @ldap_pam = exported_conf['pam']
+                @ldap_pam = false if @ldap_pam.nil?
+                @ldap_nss = exported_conf['nss']
+                @ldap_nss = [] if @ldap_nss.nil?
+            end
         end
 
         # Generate ldap.conf content from the current configuration.
@@ -510,7 +516,7 @@
             if @ldap_pam
                 pkgs += ['pam_ldap']
             end
-            if @ldap_nss
+            if @ldap_nss.any?
                 pkgs += ['nss_ldap']
                 if @ldap_nss.include?('automount')
                     pkgs += ['openldap2-client'] # provides 
/etc/openldap/ldap.conf
@@ -666,10 +672,16 @@
 
         # Set configuration for Kerberos from exported objects.
         def krb_import(exported_conf)
-            @krb_conf = exported_conf['conf']
+            if exported_conf.nil?
+                @krb_conf = {}
+                @krb_pam = false
+            else
+                @krb_conf = exported_conf['conf']
+                @krb_conf = {} if @krb_conf.nil?
+                @krb_pam = exported_conf['pam']
+                @krb_pam = false if @krb_pam.nil?
+            end
             krb_lint_conf
-            @krb_pam = exported_conf['pam']
-            @krb_pam = false if @krb_pam.nil?
         end
 
         # Make sure the Kerberos configuration has all the necessary keys.
@@ -818,12 +830,18 @@
 
         # Set configuration for auxiliary daemons/PAM from exported objects.
         def aux_import(exported_conf)
-            @autofs_enabled = exported_conf['autofs']
-            @autofs_enabled = false if @autofs_enabled.nil?
-            @nscd_enabled = exported_conf['nscd']
-            @nscd_enabled = false if @nscd_enabled.nil?
-            @mkhomedir_pam = exported_conf['mkhomedir']
-            @mkhomedir_pam = false if @mkhomedir_pam.nil?
+            if exported_conf.nil?
+                @autofs_enabled = false
+                @nscd_enabled = false
+                @mkhomedir_pam = false
+            else
+                @autofs_enabled = exported_conf['autofs']
+                @autofs_enabled = false if @autofs_enabled.nil?
+                @nscd_enabled = exported_conf['nscd']
+                @nscd_enabled = false if @nscd_enabled.nil?
+                @mkhomedir_pam = exported_conf['mkhomedir']
+                @mkhomedir_pam = false if @mkhomedir_pam.nil?
+            end
         end
 
         # Immediately enable(start)/disable(stop) the auxiliary daemons.
@@ -939,12 +957,21 @@
 
         # Set configuration for AD enrollment from exported objects.
         def ad_import(exported_conf)
-            @ad_domain = exported_conf['domain']
-            @ad_user = exported_conf['user']
-            @ad_ou = exported_conf['ou']
-            @ad_pass= exported_conf['pass']
-            @ad_overwrite_smb_conf = exported_conf['overwrite_smb_conf']
-            @ad_update_dns = exported_conf['update_dns']
+            if exported_conf.nil?
+                @ad_domain = ''
+                @ad_user = ''
+                @ad_ou = ''
+                @ad_pass = ''
+                @ad_overwrite_smb_conf = false
+                @ad_update_dns = false
+            else
+                @ad_domain = exported_conf['domain']
+                @ad_user = exported_conf['user']
+                @ad_ou = exported_conf['ou']
+                @ad_pass= exported_conf['pass']
+                @ad_overwrite_smb_conf = exported_conf['overwrite_smb_conf']
+                @ad_update_dns = exported_conf['update_dns']
+            end
         end
 
         # Run "net ads join". Return tuple of boolean success status and 
command output.
@@ -1049,7 +1076,7 @@
             if @krb_pam
                 pkgs += ['pam_krb5', 'krb5', 'krb5-client']
             end
-            if @ldap_nss
+            if @ldap_nss.any?
                 pkgs += ['nss_ldap']
                 if @ldap_nss.include?('automount')
                     pkgs += ['openldap2-client'] # provides 
/etc/openldap/ldap.conf

Reply via email to