[Authenticator::Base] case_insensitive_find_by_userid cache

ManageIQ · Sep 2, 2020 · 06c71cc · 06c71cc
1 parent b1bc828
commit 06c71cc
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/app/models/authenticator/base.rb b/app/models/authenticator/base.rb
@@ -244,9 +244,13 @@ def failure_reason(_username, _request)
  end
 
  def case_insensitive_find_by_userid(username, lookup_scope: nil)
+ @lookup ||= {}
+ @lookup[username] ||= {}
+ return @lookup[username][lookup_scope] if @lookup[username][lookup_scope]
+
  base = User.send(User::LOOKUP_SCOPES[lookup_scope])
  user = base.lookup_by_userid(username)
- user || base.in_my_region.where('lower(userid) = ?', username.downcase).order(:lastlogon).last
+ @lookup[username][lookup_scope] = user || base.in_my_region.where('lower(userid) = ?', username.downcase).order(:lastlogon).last
  end
 
  def userid_for(_identity, username)