Issue #19056 has been updated by Andrew Parker.
Released in 3.3.0 ---------------------------------------- Bug #19056: AIX user provider do handle well trailling space after the password hash in local file file /etc/security/password https://projects.puppetlabs.com/issues/19056#change-99546 * Author: Stéphane Nsakala * Status: Closed * Priority: Normal * Assignee: * Category: * Target version: 3.3.0 * Affected Puppet version: 3.0.2 * Keywords: aix user password space * Branch: https://github.com/puppetlabs/puppet/pull/1763 ---------------------------------------- ! I found on my system trailing space after the hashed password. [root@test3]/root# head /etc/security/passwd root: password = 8V.f3zDkRGa7A lastupdate = 1360050752 [root@test3]/root# grep "password = 8" /etc/security/passwd | od -c 0000000 \t p a s s w o r d = 8 V . f 0000020 3 z D k R G a 7 A 0000040 \n 0000045 ! Each time i run puppet it will update the password (i did not change it between invocation. [root@test3]/root# puppet agent --test Info: Retrieving plugin Info: Caching catalog for test3.svr.luxair Info: Applying configuration version '1359997672' Notice: /Stage[main]/User::Root/User[root]/password: changed password Notice: Finished catalog run in 2.39 seconds [root@test3]/root# puppet agent --test Info: Retrieving plugin Info: Caching catalog for test3.svr.luxair Info: Applying configuration version '1359997672' ! The problem comf form the file lib/puppet/provider/user/aix.rb # If the password= entry is found, return it if l =~ /^\s*password\s*=\s*(.*)$/ password = $1; break; end ! i just printed the password f.close() print "#" + password + "#" return password ! We can see that the reg exp will not discard space after the password. So the comparison fail ! [root@test3]/root# puppet agent --test Info: Retrieving plugin Info: Caching catalog for test3.svr.luxair Info: Applying configuration version '1359997672' #8V.f3zDkRGa7A #Notice: /Stage[main]/User::Root/User[root]/password: changed password Notice: Finished catalog run in 2.39 seconds ! changing the reg exp will solve the problem *** /opt/freeware/lib/ruby/gems/1.9.1/gems/puppet-3.0.2/lib/puppet/provider/user/aix.rb Tue Feb 5 09:07:11 2013 --- /tmp/aix.rb.new Tue Feb 5 09:06:48 2013 *************** *** 235,241 **** # If there is a new user stanza, stop break if l =~ /^\S*:\s*$/ # If the password= entry is found, return it ! if l =~ /^\s*password\s*=\s*(.*)$/ password = $1; break; end } --- 235,241 ---- # If there is a new user stanza, stop break if l =~ /^\S*:\s*$/ # If the password= entry is found, return it ! if l =~ /^\s*password\s*=\s*(\S*)\s*$/ password = $1; break; end } -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/groups/opt_out.
