rails-activestorage - 如何处理 ActiveStorage 直接上传的 ActiveSupport::MessageVerifier::InvalidSignature 错误

Question

我有一个表单，用户可以在其中选择要上传的文件。一旦使用 ActiveStorage 直接上传选择了文件，就会上传该文件。有一个提交按钮，在上传完成之前不可点击。

该按钮在表单加载时被禁用。但是，如果由于某种原因用户能够在没有先选择文件的情况下单击按钮，我的 Controller 中会出现以下错误:

ActiveSupport::MessageVerifier::InvalidSignature - ActiveSupport::MessageVerifier::InvalidSignature:

我想确保我的应用程序是可靠的，所以我想知道如何处理这个错误。

这是我的 Controller 方法:

def create
  authorize [:proofreaders, :proofread_document]
  @proofread_document = ProofreadDocument.build(proofread_document_params.merge(uploader: current_user, proofreading_job: @proofreading_job))
  if @proofread_document.save
    flash[:notice] = I18n.t('success.upload', resource: @proofread_document.file.filename)
    render :create, layout: false
  else
    render :new, layout: false
  end
end

我处理这个问题的尝试是创建我自己的“build”方法并使用它而不是标准的“new”，因为“new”在我的 Controller 中引发了错误，我想在模型中处理这个问题。

我的模型中的“构建”方法:

  def self.build(params)
    pd = self.new params.except(:file)
    pd.file = params[:file]
    pd
  rescue ActiveSupport::MessageVerifier::InvalidSignature => e
    pd
  end

我必须单独将文件属性值分配给模型，以便在引发错误之前分配其他属性。

这有效，因为它返回在错误发生之前设置的模型实例，并且文件验证检查空文件值。但是，这真的正确吗？是否有更好的方法供更有经验和技能的开发人员使用？

Answer 1

我知道如何解决这个问题的方法是修改参数以拒绝空白值。例如

params[:user].delete(:avatar) if params[:user][:avatar].blank?

这有效，但它会使您的 Controller 操作更难查看我想在模型方面解决这个问题，所以我会这样做并填写结果如何。