diff options
author | Matt Robinson <matt@puppetlabs.com> | 2010-05-17 15:00:52 -0700 |
---|---|---|
committer | test branch <puppet-dev@googlegroups.com> | 2010-02-17 06:50:53 -0800 |
commit | cce63d86d7ce3965e58d8e57ff19533a3f21b4eb (patch) | |
tree | 59d88972e46a1a1f1d78d6291a78704fd2ad2c33 /lib/puppet | |
parent | 4ba3dc22fe4c6b54564d468e70b79e4695911c61 (diff) | |
download | puppet-cce63d86d7ce3965e58d8e57ff19533a3f21b4eb.tar.gz puppet-cce63d86d7ce3965e58d8e57ff19533a3f21b4eb.tar.xz puppet-cce63d86d7ce3965e58d8e57ff19533a3f21b4eb.zip |
Bug #3748 LDAP group membership
LDAP group membership comparison was happening on an unsorted string.
Sorting the string for now, may want to do something smarter by
comparing something other than strings later.
Signed-off-by: Matt Robinson <matt@puppetlabs.com>
Diffstat (limited to 'lib/puppet')
-rw-r--r-- | lib/puppet/provider/user/ldap.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/puppet/provider/user/ldap.rb b/lib/puppet/provider/user/ldap.rb index 4ac18030a..7c38880cb 100644 --- a/lib/puppet/provider/user/ldap.rb +++ b/lib/puppet/provider/user/ldap.rb @@ -65,7 +65,7 @@ Puppet::Type.type(:user).provide :ldap, :parent => Puppet::Provider::Ldap do return @property_hash[:groups] = :absent end - return @property_hash[:groups] = result.collect { |r| r[:name] }.join(",") + return @property_hash[:groups] = result.collect { |r| r[:name] }.sort.join(",") end return @property_hash[:groups] end |