Question

今天，每次我开始delayed_job工人时，这个过程都会立刻死去。

经过一些调查（并找到delayed_job的前台模式），我终于发现问题是delayed_job序列化我的活动记录对象的方式是在YAML加载时触发异常部分：

Psych::SyntaxError: (<unknown>): mapping keys are not allowed in this context at line 7 column 14
from /Users/mick/.rvm/rubies/ruby-1.9.3-p448/lib/ruby/1.9.1/psych.rb:203:in `parse'
from /Users/mick/.rvm/rubies/ruby-1.9.3-p448/lib/ruby/1.9.1/psych.rb:203:in `parse_stream'
from /Users/mick/.rvm/rubies/ruby-1.9.3-p448/lib/ruby/1.9.1/psych.rb:151:in `parse'
from /Users/mick/.rvm/rubies/ruby-1.9.3-p448/lib/ruby/1.9.1/psych.rb:127:in `load'
from /Users/mick/.rvm/gems/ruby-1.9.3-p448/gems/safe_yaml-0.9.7/lib/safe_yaml.rb:144:in `load_with_options'
from (irb):111
from /Users/mick/.rvm/gems/ruby-1.9.3-p448/gems/railties-3.2.16/lib/rails/commands/console.rb:47:in `start'
from /Users/mick/.rvm/gems/ruby-1.9.3-p448/gems/railties-3.2.16/lib/rails/commands/console.rb:8:in `start'
from /Users/mick/.rvm/gems/ruby-1.9.3-p448/gems/railties-3.2.16/lib/rails/commands.rb:41:in `<top (required)>'
from script/rails:6:in `require'
from script/rails:6:in `<main>'

delayed_job尝试时发生的事情：

YAML.load(my_job.handler)

（others had the same issue before me）

找到有问题的Delayed::Backend::ActiveRecord::Job个实例后，puts my_job.handler会显示：

object: !ruby/ActiveRecord:MyActiveRecord
  attributes:
    id: 7648
    ... some good stuff ...
    my_field: ?   bla bla bla
    ... some other good stuff ...
method_name: :mail
args: []

我首先认为这是一个编码问题，但我意识到'？'性格是一个真正的'？'字符（即值63）而不是对未识别字符的误解。

然后我尝试创建一个my_field值为? Totot的活动记录类的新实例，但是YAML看起来如下：

object: !ruby/ActiveRecord:MyActiveRecord
  attributes:
    id: 7648
    ... some good stuff ...
    my_field: ! '?   bla bla bla'
    ... some other good stuff ...
method_name: :mail
args: []

并且YAML.load(...)成功运行。

所以我的问题是：

知道我的数据库中有多乱的YAML吗？
知道我应该如何消毒我的params来避免这样的问题？
知道如何在单元测试中重现这一点吗？（可以肯定的是我实际上正在使用第2步修复它）

Answer 1

@ house9建议的详细解释：

请勿执行以下操作（即使delayed_job的git repo建议为例）

Notifier.delay.signup(@user)

class NotifierMailer < ActionMailer::Base
  def signup(user)
  end
end

因为这将尝试yaml编码@user（这可能会导致问题）

但是，每当你有一个具有id的对象（尤其是AR对象）时，你应该在调用延迟的作业时传递id并在以后检索它：

Notifier.delay.signup(@user.id)

class NotifierMailer < ActionMailer::Base
  def signup(id)
     @user = User.find_by_id(id)
  end
end

Answer 2

知道我应该如何消毒我的params以避免这样的问题？

不要序列化您的activerecord对象，而只是序列化AR id，然后执行find作为作业的第一步。

不确定为什么序列化AR已损坏，您的架构是否在序列化发生之前和作业运行之间发生了变化？

如何重现/清理凌乱的POST参数以避免使用delayed_job的YAML序列化问题？

2 个答案: