Re:Never Give Your Information To 10 Minute Old Startups
by oelmekki on Saturday October 06, 2012 @06:15AM
(#4620218)
> it's somewhat understandable in the context of a site that unintentionally goes public before it's ready
Sorry, but no. There is absolutely no good reason while creating an user controller - even when writing the very first lines - to not check against current user on sensible actions.
Actually, I may argue that's the very first thing you do once the controller skeleton is set up.
Re:Never Give Your Information To 10 Minute Old Startups
by alinajaf on Saturday October 06, 2012 @07:24AM
(#4620275)
Agreed. You don't even need to add any gems or do any rails magic for this to work. In your controller:
def edit
@aws_credentials = current_user.aws_credentials
end
def update
current_user.aws_credentials.update_attributes(
params[:aws_credentials].slice(:a, :b, :c)
)
end
They should have thanked him, notified their users, done a thorough review of their own security, and warned new signups to only use IAM keys. Instead they got defensive, made excuses, and attacked the messenger.