ldap_fluff 0.4.0 → 0.4.1
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/lib/ldap_fluff/generic.rb +1 -2
- data/test/ad_test.rb +24 -0
- metadata +4 -2
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: 161e33a4c78a2bb8405942c777279117099659cd
|
4
|
+
data.tar.gz: 668e42def5a195c6ca935b2c7dcebe63ad96bf9b
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: ae3b5e9be642e485cf2713d3f318985f891f9651a5ed3b069a572be045e26ae3398135093fe63929dc96fcd450637f216cd3440f22bee5ed94c48056afca2d45
|
7
|
+
data.tar.gz: 0ce4d8b5eb2b3ef58df3d314e41188503daa403cd14f078215a11c9f526662da9d3d59aa29b4a85dd90f4520b0257484f4735f2e10d03586577e6395ddacf40b
|
data/lib/ldap_fluff/generic.rb
CHANGED
@@ -43,8 +43,7 @@ class LdapFluff::Generic
|
|
43
43
|
return [] unless group_exists?(gid)
|
44
44
|
search = @member_service.find_group(gid).last
|
45
45
|
|
46
|
-
method = [:member, :
|
47
|
-
:memberuid, :uniquemember].find { |m| search.respond_to? m } or
|
46
|
+
method = [:member, :memberuid, :uniquemember].find { |m| search.respond_to? m } or
|
48
47
|
return []
|
49
48
|
|
50
49
|
users_from_search_results(search, method)
|
data/test/ad_test.rb
CHANGED
@@ -172,4 +172,28 @@ class TestAD < MiniTest::Test
|
|
172
172
|
md.verify
|
173
173
|
end
|
174
174
|
|
175
|
+
def test_find_users_with_empty_nested_group
|
176
|
+
group = Net::LDAP::Entry.new('foremaners')
|
177
|
+
nested_group = Net::LDAP::Entry.new('katellers')
|
178
|
+
nested_user = Net::LDAP::Entry.new('testuser')
|
179
|
+
|
180
|
+
group[:member] = ['CN=Test User,CN=Users,DC=corp,DC=windows,DC=com', 'CN=katellers,DC=corp,DC=windows,DC=com']
|
181
|
+
nested_group[:cn] = ['katellers']
|
182
|
+
nested_group[:objectclass] = ['organizationalunit']
|
183
|
+
nested_group[:memberof] = ['CN=foremaners,DC=corp,DC=windows,DC=com']
|
184
|
+
nested_user[:objectclass] = ['person']
|
185
|
+
|
186
|
+
md = MiniTest::Mock.new
|
187
|
+
2.times { md.expect(:find_group, [group], ['foremaners']) }
|
188
|
+
2.times { md.expect(:find_group, [nested_group], ['katellers']) }
|
189
|
+
2.times { service_bind }
|
190
|
+
|
191
|
+
md.expect(:find_by_dn, [nested_user], ['CN=Test User,CN=Users,DC=corp,DC=windows,DC=com'])
|
192
|
+
md.expect(:find_by_dn, [nested_group], ['CN=katellers,DC=corp,DC=windows,DC=com'])
|
193
|
+
md.expect(:get_login_from_entry, 'testuser', [nested_user])
|
194
|
+
@ad.member_service = md
|
195
|
+
assert_equal @ad.users_for_gid('foremaners'), ['testuser']
|
196
|
+
md.verify
|
197
|
+
end
|
198
|
+
|
175
199
|
end
|
metadata
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: ldap_fluff
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.4.
|
4
|
+
version: 0.4.1
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Jordan O'Mara
|
@@ -9,10 +9,11 @@ authors:
|
|
9
9
|
- Petr Chalupa
|
10
10
|
- Adam Price
|
11
11
|
- Marek Hulan
|
12
|
+
- Dominic Cleal
|
12
13
|
autorequire:
|
13
14
|
bindir: bin
|
14
15
|
cert_chain: []
|
15
|
-
date:
|
16
|
+
date: 2016-03-03 00:00:00.000000000 Z
|
16
17
|
dependencies:
|
17
18
|
- !ruby/object:Gem::Dependency
|
18
19
|
name: net-ldap
|
@@ -91,6 +92,7 @@ email:
|
|
91
92
|
- pchalupa@redhat.com
|
92
93
|
- komidore64@gmail.com
|
93
94
|
- mhulan@redhat.com
|
95
|
+
- dominic@cleal.org
|
94
96
|
executables: []
|
95
97
|
extensions: []
|
96
98
|
extra_rdoc_files:
|