Ruby on Rails - 权威人士 - 控制器的未知操作

Question

这个问题指的是this具体问题。

我使用 pundit 作为我的授权 gem，我希望用户 X 只能下载属于用户 X 的用户信息。现在我有 http://localhost:3000/download.csv 作为我的链接来接收用户信息。但是如果我登录到另一个用户，例如 user/2，我仍然可以输入 url 并下载 user/3 数据。

我现在拥有的：

user_policy.rb

class UserPolicy < ApplicationPolicy

  def profile?
    true
  end

  def download?

  end

  private

  def user_or_admin
    user.id == record.id || user.admin?
  end


end

application_policy.rb

class ApplicationPolicy
  attr_reader :user, :record

  def initialize(user, record)
    @user = user
    @record = record
  end

  def index?
    false
  end

  def show?
    scope.where(:id => record.id).exists?
  end

  def create?
    create?
  end

  def new?
    create?
  end

  def update?
    false
  end

  def edit?
    update?
  end

  def destroy?
    false
  end

  class Scope
    attr_reader :user, :scope

    def initialize(user, scope)
      @user = user
      @scope = scope
    end

    def resolve
      scope.all
    end
  end
end

这是我的 user_controller.rb # 这不是 users_controller，那是不同的视图

    class UserController < ApplicationController
  prepend_view_path(File.join(Rails.root, 'app/views/user/'))


  layout 'application'


  def index

  end

  def billing
  end

  def plan
  end

  def profile
    @user = User.find(params[:id])
    @user_posts = @user.posts.order('created_at DESC')
  end
end

def download
  @user = User.find(params[:id])

  respond_to do |format|
    format.html
    format.csv { send_data @user.csv, filename: "userinfo-#{Date.today}.csv" }
  end
end

  def support
    @user = User.find(params[:id])
    @user_posts = @user.posts.order('created_at DESC')
  end

  def notifications
  end

  def show
    @user = User.find(params[:id])
    @posts = current_user.posts.order('created_at DESC')
    @user_posts = @user.posts.order('created_at DESC')
  end

更新：按照建议，我尝试在我的 user_controller 中实现新的下载操作，并尝试在我的视图中使用此代码：

<p><%= link_to("Export data as CSV", download_path(@user, format: :csv), { :controller => :user, :action => :download }, class: "btn btn-success") %></p> 

但这会引发以下错误：

接收未知操作 - 找不到 UserController 的操作“下载”

routes.rb

        Rails.application.routes.draw do

  get 'legal/privacy'

  get :datenschutz, to: 'legal#terms_of_service'


  devise_for :users, path_names: { sign_in: 'login', sign_out: 'logout', sign_up: 'registrieren', edit: 'bearbeiten' }, :controllers => { registrations: 'registrations' }

  get '/users/mitteilungen/:id' => 'user#notifications'
  get '/users/:id/artikel/', :to => 'user#support', :as => :artikel
  get '/users/plan' => 'user#plan'
  get '/users/billing' => 'user#billing'
  get '/users/:id', :to => 'user#profile', :as => :user
  get 'download', :to => 'user#download', :controller => :user, action: :download, :as => :download


  resources :posts, path: :news do
    resources :comments, path: :kommentare do
    end
  end


  devise_scope :user do
    resources :posts, path: :news do
      resources :comments do

      end
    end
  end
  root to: "posts#index", as: :root

end

Answer 1

你的条件不对@user.id = user.id。

user 是@user。我想你的意思是user.id = record.id

record 是您正在授权的实例，例如：

def show
  @user = User.find(params[:id])
  authorize @user
  ...
end

authorize @user 基本上是：UserPolicy.new(@user, current_user).show? 的快捷方式

顺便说一句，我不确定，但我认为您可以放心地省略 .id: user == record

UPD。 我重新阅读了您的政策。 ApplicationPolicy 通常如下所示：

class ApplicationPolicy
  attr_reader :user, :record

  def initialize(user, record)
    @user = user
    @record = record
  end
end

你重写了构造函数。为什么？让它保持 Pundit 想要的样子

UPD2。这就是我希望您的 UserPolicy 的样子：

class UserPolicy < ApplicationPolicy
  def profile? #this is the action I have the button in to export data
    return true if user_or_admin?
  end

  private

  def user_or_admin?
    user.id == record.id || user.admin?
  end
end

Answer 2

我们在 cmets 中对我的原始答案进行了太多讨论。我想澄清一下我的观点。

你写道：

我使用 pundit 作为我的授权 gem，我希望用户 X 只能下载属于用户 X 的用户数据。现在我有 http://localhost:3000/download/x.csv 作为我的链接来接收用户数据。但是如果我登录到另一个用户，例如 user/:id 2，我仍然可以输入 url 并下载 users/3 数据。

所以问题是授权不起作用。我说的对吗？

之后你写道：

更新：按照建议，我尝试在我的 user_controller 并尝试在我看来使用此代码： ...

但这会引发以下错误： ...

因此，您的路由问题超出了原始问题的范围，应该在不同的问题中解决。无论如何，您可以查看this question。

所以。如果您的问题是授权 - 只需正确编写策略并在控制器中使用它们。就这些了。