Bug #17441

"Puppet class not found" when trying to override all parameters of a class as a user

Added by Ori Rabin over 1 year ago. Updated over 1 year ago.

Assigned To:Ori Rabin
Target version:-
Difficulty: Bugzilla link:1390312
Found in release: Pull request:https://github.com/theforeman/foreman/pull/4041
Story points-
Velocity based estimate-
Release1.13.3Release relationshipAuto


Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1390312

Description of problem:
When using a user account with limited rights (e.g. only view and edit puppetclasses), the user cannot click the "Override all parameters" button, as it will lead to a 404 error.

Version-Release number of selected component (if applicable):
Satellite 6.2.3

How reproducible:

Steps to Reproduce:
1. create a user with limited rights
2. grant the user the "view_puppetclasses" and "edit_puppetclasses" permissions
3. open https://sat/puppetclasses as that user, click "Override all parameters" for a class

Actual results:
"Puppet class not found Please try to update your request"

2016-10-28 15:54:49 [app] [I] Started POST "/puppetclasses/9-test/override?enable=true" for at 2016-10-28 15:54:49 +0200
2016-10-28 15:54:49 [app] [I] Processing by PuppetclassesController#override as HTML
2016-10-28 15:54:49 [app] [I] Parameters: {"authenticity_token"=>"TuAomLSSXoW9R+2caJKH/Ob5pu82ZcFXwBFuDPzV+A8=", "enable"=>"true", "id"=>"9-test"}
2016-10-28 15:54:49 [app] [I] Rendered common/404.html.erb within layouts/application (0.9ms)
2016-10-28 15:54:49 [app] [I] Rendered layouts/_application_content.html.erb (0.2ms)
2016-10-28 15:54:49 [app] [I] Rendered layouts/base.html.erb (0.8ms)
2016-10-28 15:54:49 [app] [I] Completed 404 Not Found in 48ms (Views: 2.9ms | ActiveRecord: 11.6ms)

Expected results:
parameters are overriden now

Additional info:

Associated revisions

Revision e18d69e8
Added by Ori Rabin over 1 year ago

Fixes #17441 - override all parameters permission changed to edit

Revision 2d4468a7
Added by Ori Rabin over 1 year ago

Fixes #17441 - override all parameters permission changed to edit

(cherry picked from commit e18d69e89b544288d90926011d7bb8b1d46a182b)


#1 Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4041 added

#2 Updated by Ori Rabin over 1 year ago

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

#3 Updated by Dominic Cleal over 1 year ago

  • Release set to 1.13.3

Also available in: Atom PDF