Project

General

Profile

Bug #4442

Attempt to use login/password authentication for AuthSourceExternal leads to traceback

Added by Jan Pazdziora almost 7 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Authentication
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

When a user is populated in AuthSourceExternal in the http://projects.theforeman.org/projects/foreman/wiki/Foreman_and_mod_auth_kerb setup, an attempt to do non-Kerberos authentication on /users/login leads to

Oops, we're sorry but something went wrong

Warning!
undefined local variable or method `__class__' for #

If you feel this is an error with Foreman itself, please open a new issue with Foreman ticketing system, You would probably need to attach the Full trace and relevant log entries.

NameError
undefined local variable or method `__class__' for #<AuthSourceExternal:0x00000005d3cac0>
app/models/auth_sources/auth_source_external.rb:4:in `authenticate'
app/models/user.rb:116:in `try_to_login'
app/controllers/users_controller.rb:63:in `login'
app/models/concerns/foreman/thread_session.rb:33:in `clear_thread'
lib/middleware/catch_json_parse_errors.rb:9:in `call'

Back

Associated revisions

Revision a416bb57 (diff)
Added by Jan Pazdziora over 6 years ago

fixes #4442 - returning nil plays nice with try_to_login.

Revision 99273ffa (diff)
Added by Jan Pazdziora over 6 years ago

fixes #4442 - returning nil plays nice with try_to_login.

(cherry picked from commit a416bb573dedf6c34592afec17fbbec96691a5e6)

History

#1 Updated by Dominic Cleal almost 7 years ago

  • Category set to Authentication
  • Legacy Backlogs Release (now unused) deleted (2)

#3 Updated by Dominic Cleal almost 7 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Jan Pazdziora
  • Target version set to 1.9.1

#4 Updated by Dmitri Dolguikh almost 7 years ago

  • Target version changed from 1.9.1 to 1.9.0

#5 Updated by Dominic Cleal over 6 years ago

  • Legacy Backlogs Release (now unused) set to 7

#6 Updated by Jan Pazdziora over 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF