Skip to content

Instantly share code, notes, and snippets.

@haikusw
Forked from dhh/gist:1975644
Created March 5, 2012 00:50
Show Gist options
  • Save haikusw/1975696 to your computer and use it in GitHub Desktop.
Save haikusw/1975696 to your computer and use it in GitHub Desktop.
class PostsController < ActionController::Base
def create
Post.create(post_params)
end
def update
Post.find(params[:id]).update_attributes!(post_params)
end
private
def post_params
params[:post].slice(:title, :content)
end
end
@haikusw
Copy link
Author

haikusw commented Mar 5, 2012

"Here's the basic pattern we've been using at 37signals to guard against mass-assignment bugs. The right place to guard against these issues is in the controller, not the model. AR's attr_accessible is the wrong way about it." - dhh

@haikusw
Copy link
Author

haikusw commented Mar 5, 2012

"Good suggestion from @nk:. Mark params as tainted and have create/update_attributes bark if not untainted via #slice and friends." -dhh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment