ldap_fluff 0.4.5 → 0.4.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 34897e06c74129a3f1bf8ac3941f0e0fe26b4b2b
-  data.tar.gz: 698fba54e97bc907f69330b57c4c20bfbffd567d
+  metadata.gz: 11ad7cdc6f2febae65680c15ad1649f54f949085
+  data.tar.gz: a34667350a400be3b50742c34b7e4d269a4e4e9c
 SHA512:
-  metadata.gz: 8091ae1ea8dd8078caab49613066a69bbe5cb3d7f0c18d40c9bc9a2a57d319b8c676b338459e8b1fc6a98f4c45f9323be099e97f3babf5917b7368a355c14940
-  data.tar.gz: 530f6e5af782e44d598a484c07ecedf3fe311768fd77df848ce8e126158fe1609316aa62c86fbbdbbce9fb9ed3d1f83e2bfbcb57279d46e0ad9de2ac44390365
+  metadata.gz: 250bdde42a707b3d253f93430e45d6979326f6ab58e67af99798b4e5a05df67f5bd1a4c1aa8888155480f16890c0ae5d9ecc1ed731cb7e60f0db9460a2f077a1
+  data.tar.gz: 1f605000dd7800eebb73a76c0ceab30cba2f039378a87a17b470c8d92ba246c61ca99928e1308017946c834f3ac19661e874aacb585dd99b0cbd505bdf028eb3

data/lib/ldap_fluff/active_directory.rb

@@ -30,7 +30,11 @@ class LdapFluff::ActiveDirectory < LdapFluff::Generic
     users = []
 
     search.send(method).each do |member|
-      entry = @member_service.find_by_dn(member).first
+      begin
+        entry = @member_service.find_by_dn(member).first
+      rescue MemberService::UIDNotFoundException
+        next
+      end
       objectclasses = entry.objectclass.map(&:downcase)
 
       if (%w(organizationalperson person userproxy) & objectclasses).present?

data/test/ad_test.rb

@@ -110,6 +110,19 @@ class TestAD < MiniTest::Test
     assert_equal(@ad.is_in_groups("john", %w(broskies), true), true)
   end
 
+  def test_subgroups_in_groups_are_ignored
+    group        = Net::LDAP::Entry.new('foremaners')
+    md = MiniTest::Mock.new
+    2.times { md.expect(:find_group, [group], ['foremaners']) }
+    2.times { service_bind }
+    def md.find_by_dn(dn)
+      raise LdapFluff::ActiveDirectory::MemberService::UIDNotFoundException
+    end
+    @ad.member_service = md
+    assert_equal @ad.users_for_gid('foremaners'), []
+    md.verify
+  end
+
   def test_user_exists
     md = MiniTest::Mock.new
     md.expect(:find_user, 'notnilluser', %w(john))

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ldap_fluff
 version: !ruby/object:Gem::Version
-  version: 0.4.5
+  version: 0.4.6
 platform: ruby
 authors:
 - Jordan O'Mara
@@ -13,62 +13,62 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-11 00:00:00.000000000 Z
+date: 2017-02-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: net-ldap
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.3.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.3.1
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Simple library for binding & group querying on top of various LDAP implementations
@@ -118,27 +118,27 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.5
+rubygems_version: 2.5.2
 signing_key: 
 specification_version: 4
 summary: LDAP querying tools for Active Directory, FreeIPA and POSIX-style
 test_files:
-- test/ipa_member_services_test.rb
-- test/posix_member_services_test.rb
-- test/ad_test.rb
-- test/ipa_test.rb
+- test/ad_member_services_test.rb
 - test/ldap_test.rb
+- test/config_test.rb
+- test/posix_member_services_test.rb
+- test/ipa_member_services_test.rb
 - test/lib/ldap_test_helper.rb
+- test/ad_test.rb
 - test/posix_test.rb
-- test/ad_member_services_test.rb
-- test/config_test.rb
+- test/ipa_test.rb