Bug #24883
closedFailed to unset host/host group's openscap_proxy_id with "undefined method `destroy' for nil:NilClass" error
Description
Description of problem:
Unset host/host group's openscap_proxy_id with missing lookup key parameter will cause the following error.
NoMethodError: undefined method `destroy' for nil:NilClass
from /opt/theforeman/tfm/root/usr/share/gems/gems/foreman_openscap-0.5.3.23/app/models/concerns/foreman_openscap/openscap_proxy_core_extensions.rb:43:in `map'
from /opt/theforeman/tfm/root/usr/share/gems/gems/foreman_openscap-0.5.3.23/app/models/concerns/foreman_openscap/openscap_proxy_core_extensions.rb:43:in `destroy_scap_client_lookup_values'
from /opt/theforeman/tfm/root/usr/share/gems/gems/foreman_openscap-0.5.3.23/app/models/concerns/foreman_openscap/openscap_proxy_core_extensions.rb:25:in `update_scap_client_params'
from /opt/theforeman/tfm/root/usr/share/gems/gems/foreman_openscap-0.5.3.23/app/models/concerns/foreman_openscap/openscap_proxy_core_extensions.rb:12:in `update_scap_client'
How reproducible:
1) Choose a host group with openscap_proxy_id set. Lets say host_group_A
2) Web UI -> Configure -> Smart Class Parameters -> search for "server" parameter.
3) In server parameter -> Under "Specify matchers" -> remove host_group_A
4) Web UI -> Configure -> Edit host_group_A -> Unset openscap_proxy_id -> Click submit
Oops, we're sorry but something went wrong undefined method `destroy' for nil:NilClass
Updated by The Foreman Bot about 6 years ago
- Status changed from New to Ready For Testing
- Assignee set to Ondřej Pražák
- Pull request https://github.com/theforeman/foreman_openscap/pull/356 added
Updated by Marek Hulán about 6 years ago
- Subject changed from Failed to unset host/host group's openscap_proxy_id with "undefined method `destroy' for nil:NilClass" error to Failed to unset host/host group's openscap_proxy_id with "undefined method `destroy' for nil:NilClass" error
- Target version set to foreman_openscap 0.11.0
- Fixed in Releases foreman_openscap 0.11.0 added
Updated by Ondřej Pražák about 6 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman_openscap|5832b7ed2bb8337b765035e694551f1fe9722078.