이것은 강력한 user_params 강력한 매개 변수 구현입니까?

Question

def user_params 
    if current_user.admin? 
    params.require(:user).permit(:name, :email, :password, :student, :admin) 
    else 
    params.require(:user).permit(:name, :email, :password) 
    end       
end 

사용자의 역할을 업데이트 할 수 있기를 바랍니다.

Users_params는 해커가 사용자의 역할을 변경하지 못하게하는 강력한 매개 변수입니다.

아래 코드는 안전하지 않습니까? GitHub 전체에서이 코드를 볼 수 있습니다.

def user_params 
    params.require(:user).permit(:name, :email, :password, :student, :admin)       
end 

Answer 1

def user_params 
    if current_user.admin? 
    params.require(:user).permit! 
    else 
    params.require(:user).permit(:name, :email, :password) 
    end       
end