Bug #5645

Undefined method session when default organization set

Added by Eric Helms about 4 years ago. Updated 8 days ago.

Status:Closed
Priority:Normal
Assignee:Eric Helms
Category:Authentication
Target version:1.5.1
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link:1096223 Found in Releases:1.5.0
Pull request:

Description

NameError: undefined local variable or method `session' for Foreman::Controller::UsersMixin:Module
 | /home/ehelms/workspace/foreman/app/controllers/concerns/foreman/controller/users_mixin.rb:60:in `block in set_current_taxonomies'
 | /home/ehelms/workspace/foreman/app/controllers/concerns/foreman/controller/users_mixin.rb:55:in `each'
 | /home/ehelms/workspace/foreman/app/controllers/concerns/foreman/controller/users_mixin.rb:55:in `set_current_taxonomies'
 | /home/ehelms/workspace/foreman/app/controllers/concerns/foreman/controller/authentication.rb:77:in `set_current_user'
 | /home/ehelms/workspace/foreman/app/controllers/concerns/foreman/controller/authentication.rb:18:in `authenticate'

Associated revisions

Revision e44762c8
Added by Eric Helms about 4 years ago

Fixes #5645: Fixes session undefined error when user has a default taxonomy set.

Revision b8571d9b
Added by Eric Helms about 4 years ago

Fixes #5645: Fixes session undefined error when user has a default taxonomy set.

(cherry picked from commit e44762c830161a70235428cf175537fa9544377e)

History

#1 Updated by Dominic Cleal about 4 years ago

  • Description updated (diff)
  • Category set to Authentication
  • Legacy Backlogs Release (now unused) set to 16

#2 Updated by Eric Helms about 4 years ago

  • Bugzilla link set to https://bugzilla.redhat.com/show_bug.cgi?id=1096223

#3 Updated by Dominic Cleal about 4 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 1.8.3

#4 Updated by Eric Helms about 4 years ago

Steps to reproduce:

1. Create a new user and attach them to an organization
2. Save the user
3. Edit the user and set a default organization
4. Create a new role and give the user some simple permission
5. Log in as that user

#5 Updated by Lukas Zapletal about 4 years ago

Update: You need to hit the API to reproduce this one.

#6 Updated by Dmitri Dolguikh about 4 years ago

  • Target version changed from 1.8.3 to 1.8.2

#7 Updated by Eric Helms about 4 years ago

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

Also available in: Atom PDF