Merge pull request #82 from github/case-insensitive-dn-compare

Compare AD DNs case-insensitively when checking group membership

Author: mtodd

lib/github/ldap/membership_validators/active_directory.rb

@@ -31,7 +31,8 @@ def perform(entry)
             attributes: ATTRS
 
           # membership validated if entry was matched and returned as a result
-          matched.map(&:dn).include?(entry.dn)
+          # Active Directory DNs are case-insensitive
+          matched.map { |m| m.dn.downcase }.include?(entry.dn.downcase)
         end
 
         # Internal: Constructs a membership filter using the "in chain"

test/membership_validators/active_directory_test.rb

@@ -123,4 +123,13 @@ def test_validates_user_in_posix_group
     validator = make_validator(%w(posix-group1))
     assert validator.perform(@entry)
   end
+
+  def test_validates_user_in_group_with_differently_cased_dn
+    validator = make_validator(%w(all-users))
+    @entry[:dn].map(&:upcase!)
+    assert validator.perform(@entry)
+
+    @entry[:dn].map(&:downcase!)
+    assert validator.perform(@entry)
+  end
 end