Merge pull request #43 from github/AD-recursive

Add group objectclass to allow AD nested group search.

Author: mtodd

github-ldap.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |spec|
   spec.name          = "github-ldap"
-  spec.version       = "1.3.4"
+  spec.version       = "1.3.5"
   spec.authors       = ["David Calavera"]
   spec.email         = ["david.calavera@gmail.com"]
   spec.description   = %q{Ldap authentication for humans}

lib/github/ldap/group.rb

@@ -12,7 +12,7 @@ class Ldap
     class Group
       include Filter
 
-      GROUP_CLASS_NAMES = %w(groupOfNames groupOfUniqueNames posixGroup)
+      GROUP_CLASS_NAMES = %w(groupOfNames groupOfUniqueNames posixGroup group)
 
       attr_reader :ldap, :entry
 

test/group_test.rb

@@ -15,6 +15,10 @@ def setup
   end
 
   def test_group?
+    assert @group.group?(%w(group))
+    assert @group.group?(%w(groupOfUniqueNames))
+    assert @group.group?(%w(posixGroup))
+
     object_classes = %w(groupOfNames)
     assert @group.group?(object_classes)
     assert @group.group?(object_classes.map(&:downcase))