Bug #11231
closedPeriod character in parameters causes 406 error
Description
If you create a global parameter with a period character, you are then unable to modify or delete it.
I haven't tested this with host-group or host-level parameters, but I suspect the behaviour will be the same.
For example, I created a global param called "net.ifnames". When I try to edit it:
Started GET "/common_parameters/net.ifnames/edit" for 10.x.x.x at 2015-07-28 10:16:30 +0100
2015-07-28 10:16:30 [F]
ActionController::RoutingError (No route matches [GET] "/common_parameters/net.ifnames/edit"):
/usr/share/gems/gems/passenger-4.0.18/lib/phusion_passenger/rack/thread_handler_extension.rb:77:in `process_request'
/usr/share/gems/gems/passenger-4.0.18/lib/phusion_passenger/request_handler/thread_handler.rb:140:in `accept_and_process_next_request'
/usr/share/gems/gems/passenger-4.0.18/lib/phusion_passenger/request_handler/thread_handler.rb:108:in `main_loop'
/usr/share/gems/gems/passenger-4.0.18/lib/phusion_passenger/request_handler.rb:441:in `block (3 levels) in start_threads'
Trying to delete it:
Started DELETE "/common_parameters/net.ifnames" for 10.x.x.x at 2015-07-28 10:17:06 +0100
2015-07-28 10:17:06 [I] Processing by CommonParametersController#destroy as
2015-07-28 10:17:06 [I] Parameters: {"authenticity_token"=>"xxxx", "id"=>"net"}
2015-07-28 10:17:06 [I] Completed 406 Not Acceptable in 7ms (ActiveRecord: 1.1ms)
2015-07-28 10:17:21 [I]
I guess either the validation should be changed to prevent me from creating a variable with a period char in it, or the backend needs to handle params with a period in the name.
Updated by Dominic Cleal over 9 years ago
- Subject changed from period character in parameters to Period character in parameters causes 406 error
- Category set to Web Interface
Updated by Tomer Brisker over 9 years ago
- Status changed from New to Assigned
- Assignee set to Tomer Brisker
Updated by The Foreman Bot over 9 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/2583 added
- Pull request deleted (
)
Updated by Dominic Cleal over 9 years ago
- Translation missing: en.field_release set to 72
Updated by Anonymous over 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset fe74835256fc0baab8daad6411d865f9e807be3c.