diff options
author | Drew Blessing <drew@gitlab.com> | 2018-01-26 01:18:15 +0300 |
---|---|---|
committer | Drew Blessing <drew@gitlab.com> | 2018-02-13 17:42:40 +0300 |
commit | b45c7dd5fecd33d1fbc9053cb353d7ed0c6d57dd (patch) | |
tree | 42e72062303e77e28fc7729b5dc214f4fb4a6e3c /lib | |
parent | 7a18675c59d46da7234dd0cdf6960b527b513a80 (diff) |
Revert problematic LDAP person validation that threw exceptions
Constructors shouldn't throw exceptions. We also learned that
different LDAP servers behave a bit unexpectedly sometimes -
returning attributes we didn't ask for, or returned attributes
with language subtypes.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/gitlab/ldap/person.rb | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/lib/gitlab/ldap/person.rb b/lib/gitlab/ldap/person.rb index e81cec6ba1a..ec41f0ea090 100644 --- a/lib/gitlab/ldap/person.rb +++ b/lib/gitlab/ldap/person.rb @@ -63,8 +63,6 @@ module Gitlab Rails.logger.debug { "Instantiating #{self.class.name} with LDIF:\n#{entry.to_ldif}" } @entry = entry @provider = provider - - validate_entry end def name @@ -115,19 +113,6 @@ module Gitlab entry.public_send(selected_attr) # rubocop:disable GitlabSecurity/PublicSend end - - def validate_entry - allowed_attrs = self.class.ldap_attributes(config).map(&:downcase) - - # Net::LDAP::Entry transforms keys to symbols. Change to strings to compare. - entry_attrs = entry.attribute_names.map { |n| n.to_s.downcase } - invalid_attrs = entry_attrs - allowed_attrs - - if invalid_attrs.any? - raise InvalidEntryError, - "#{self.class.name} initialized with Net::LDAP::Entry containing invalid attributes(s): #{invalid_attrs}" - end - end end end end |