Bug #16269

Move to strong-params

Added by Kedar Bidarkar over 1 year ago. Updated about 1 year ago.

Status:Closed
Priority:Normal
Assigned To:Daniel Lobato Garcia
Category:-
Target version:Foreman - Team Daniel - iteration 3
Difficulty: Pull request:
Bugzilla link:
Story points-
Velocity based estimate-

Description

Currently installing the "foreman-azure" package on RHEL7 throws the below trace-back.

ForemanAzure::Azure is using attr_accessible so must either be converted to strong parameters, or add the protected_attributes gem (RuntimeError)
/usr/share/foreman/config/initializers/active_record_extensions.rb:21:in `attr_accessible'
/usr/share/foreman/app/models/concerns/optional_attr_accessible.rb:30:in `attr_accessible'
/opt/theforeman/tfm/root/usr/share/gems/gems/foreman_azure-1.0.1/app/models/foreman_azure/azure.rb:5:in `<class:Azure>'
/opt/theforeman/tfm/root/usr/share/gems/gems/foreman_azure-1.0.1/app/models/foreman_azure/azure.rb:2:in `<module:ForemanAzure>'
/opt/theforeman/tfm/root/usr/share/gems/gems/foreman_azure-1.0.1/app/models/foreman_azure/azure.rb:1:in `<top (required)>'
/opt/rh/rh-ruby22/root/usr/share/rubygems/rubygems/core_ext/kernel_require.rb:54:in `require'
/opt/rh/rh-ruby22/root/usr/share/rubygems/rubygems/core_ext/kernel_require.rb:54:in `require'
/opt/rh/sclo-ror42/root/usr/share/gems/gems/polyglot-0.3.4/lib/polyglot.rb:65:in `require'

1) Installed setup using katello-deploy and RHEL7 UI Was accessible with latest nightly.
2) Installed Azure CR via "yum install foreman-azure"
3) Restarted the katello-service and found the above issue related to Foreman:Azure

foreman-debug-zecSi.tar.xz (566 KB) Kedar Bidarkar, 08/24/2016 11:13 AM

History

#1 Updated by Kedar Bidarkar over 1 year ago

I mean, Currently installing the "foreman-azure" package on RHEL7 and then accessing the UI throws the below trace-back.

#2 Updated by Daniel Lobato Garcia over 1 year ago

  • Project changed from Foreman to Azure
  • Subject changed from Installing foreman-azure package on RHEL7 breaks the UI access to Move to strong-params
  • Category deleted (Compute resources)
  • Target version set to Team Daniel backlog

This is most likely because the RHEL6 installation you have is not nightly, but RHEL7 is (we don't have nightlies for 6). I'll fix it, it's a problem with strong params

#3 Updated by Daniel Lobato Garcia about 1 year ago

  • Status changed from New to Closed
  • Assigned To set to Daniel Lobato Garcia

24cc09c812cef7617a5b3015b9f95a8170094e83 fixed this

#4 Updated by Daniel Lobato Garcia about 1 year ago

  • Target version changed from Team Daniel backlog to Team Daniel - iteration 3

Also available in: Atom PDF