Bug #23782

No route matches {:action=>"edit", :controller=>"users", :id=>4}

Added by Shod Dan about 1 month ago. Updated 8 days ago.

Status:Closed
Priority:Normal
Assignee:Marek Hulán
Category:-
Target version:1.17.2
Difficulty:easy Team Backlog:
Triaged:Yes Fixed in Releases:1.17.2, 1.18.0
Bugzilla link: Found in Releases:1.17.0
Pull request:https://github.com/theforeman/foreman/pull/5773, https://github.com/theforeman/foreman/pull/5774, https://github.com/theforeman/foreman/pull/5767

Description

Unable to open web interface with :login: false in /etc/foreman/settings.yaml, after upgrade from 1.16 to 1.17 version.

"Oops, we're sorry but something went wrong No route matches {:action=>"edit", :controller=>"users", :id=>4}" where id 4 is admin default account.

trace.txt Magnifier (19.6 KB) Shod Dan, 06/01/2018 03:48 PM

Associated revisions

Revision 7949aadb
Added by Marek Hulán 15 days ago

Fixes #23782 - restore login disabling

History

#1 Updated by Tomer Brisker about 1 month ago

  • Legacy Backlogs Release (now unused) changed from 297 to 360

#2 Updated by Tomer Brisker 16 days ago

  • Triaged set to No
  • Difficulty set to easy

#3 Updated by Tomer Brisker 16 days ago

  • Triaged changed from No to Yes

#4 Updated by The Foreman Bot 15 days ago

  • Assignee set to Marek Hulán
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/5767 added

#5 Updated by Ohad Levy 15 days ago

do we know which commit caused it?

#6 Updated by Marek Hulán 15 days ago

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

#7 Updated by Marek Hulán 15 days ago

Most likely this one https://github.com/theforeman/foreman/pull/4973/files#diff-a0f979972c2d8e4fcb57ff902e87b928R15

perhaps the fix should hide only the user_menu part of side_menu. The email part of the fix was introduced by https://github.com/theforeman/foreman/pull/3030 but the 500 was reproducible only on settings page

#8 Updated by The Foreman Bot 15 days ago

  • Pull request https://github.com/theforeman/foreman/pull/5773 added

#9 Updated by The Foreman Bot 15 days ago

  • Pull request https://github.com/theforeman/foreman/pull/5774 added

#10 Updated by Marek Hulán 8 days ago

  • Fixed in Releases 1.17.2, 1.18.0 added

Also available in: Atom PDF