Project

General

Profile

Actions

Feature #15741

closed

Use parameter_filter instead of attr_accessible

Added by David Davis over 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
API
Target version:
Difficulty:
hard
Triaged:
Fixed in Releases:
Found in Releases:

Description

Foreman is dropping support for protected_attributes and we need to switch to their new way of whitelisting attributes. See here for more info:

http://projects.theforeman.org/projects/foreman/wiki/Strong_parameters#Changing-core-extensions-to-use-the-plugin-API

Here's the Foreman PR:

https://github.com/theforeman/foreman/pull/3659


Related issues 3 (0 open3 closed)

Related to Foreman - Feature #3917: Add strong_parameters to foremanClosedDominic Cleal12/19/2013Actions
Has duplicate Katello - Bug #5158: Remove attr_accessible from KatelloDuplicate04/11/2014Actions
Has duplicate Katello - Bug #16109: db_seed fails with Host::Managed is using attr_accessible so must either be converted to strong parameters, or add the protected_attributes gemDuplicate08/15/2016Actions
Actions #1

Updated by David Davis over 7 years ago

  • Project changed from Foreman to Katello
Actions #2

Updated by David Davis over 7 years ago

  • Category changed from API to API
Actions #3

Updated by David Davis over 7 years ago

  • Related to Bug #3659: "no implicit conversion of URI::HTTP into String" running rake reports:summarize added
Actions #4

Updated by David Davis over 7 years ago

  • Related to deleted (Bug #3659: "no implicit conversion of URI::HTTP into String" running rake reports:summarize)
Actions #5

Updated by David Davis over 7 years ago

  • Related to Feature #3917: Add strong_parameters to foreman added
Actions #6

Updated by David Davis over 7 years ago

  • Assignee deleted (David Davis)
Actions #7

Updated by David Davis over 7 years ago

One option would be to add protected_attributes as a dependency for Katello but I'd probably recommend against it. We use strong_params pretty much everywhere--except when extending the foreman models. We call attr_accessible in about 8 places that I think we can switch over to parameter_filter. I am going to investigate that route.

Actions #8

Updated by The Foreman Bot over 7 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to David Davis
  • Pull request https://github.com/Katello/katello/pull/6194 added
Actions #9

Updated by David Davis over 7 years ago

  • Has duplicate Bug #5158: Remove attr_accessible from Katello added
Actions #10

Updated by Justin Sherrill over 7 years ago

  • translation missing: en.field_release set to 162
  • Difficulty set to hard
Actions #11

Updated by Anonymous over 7 years ago

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

Updated by David Davis over 7 years ago

  • Has duplicate Bug #16109: db_seed fails with Host::Managed is using attr_accessible so must either be converted to strong parameters, or add the protected_attributes gem added
Actions

Also available in: Atom PDF