rails json.parse“意想不到的令牌”错误

Question

我有一个由Webhooks击中的简单控制器。我需要将发送的所有数据存储在模型的元数据中，该数据是text列以供以后消耗。

class NotificationsController < ApplicationController
  def create
    notification = Notification.new(
      metadata: params,
    )
    if notification.save
      head :ok
    end
  end
end

当我在任何控制器操作中检查params.class时，我会得到actionController ::参数像哈希一样的对象。

但是，当在元数据中存储params，如上所示，我得到了一个看起来像这样的字符串：

"{\"SmsSid\"=>\"ID\", \"SmsStatus\"=>\"STATUS\",\"controller\"=>\"notifications\", \"action\"=>\"create\"}"

我尝试将该字符串转换回Hash，但是执行json.parse（params）引发以下错误：

JSON::ParserError: 783: unexpected token at "{\"SmsSid\"=>\"ID\", \"SmsStatus\"=>\"STATUS\",\"controller\"=>\"notifications\", \"action\"=>\"create\"}"

这是因为列类型是text而不是jsonb？如果是这样，是否有任何不涉及数据库迁移以更改列类型的解决方法？

Answer 1

当我在任何控制器操作中检查参数时，我会得到一个actioncontroller ::参数对象，就像哈希一样。

是的，这是params是什么。这是一个对象，在大多数方面都像hash。

但是，当在元数据中存储参数时，如上所示，我会得到一个看起来像这样的字符串：

 “ {{\” smssid \“ =＆gt; \” id \“，\” smsstatus \“ =＆gt; \ \” status \ \“ status \”，\“ controller \” =＆gt; \ \ \“ notifications \”， \“ action \” =＆gt; \“ create \”}“}”
 

是的，因为您要栏杆采取不像字符串的东西，然后将其转换为字符串，以便可以将其存储在文本列。 Rails通过在对象上调用.to_s来完成此操作，该对象返回您在此处看到的字符串表示。

我尝试将该字符串转换回哈希，但是执行json.parse（params）会引发以下错误：

该字符串不是JSON。如果要序列化params hash to json，则可以使用序列化api 定义列应如何序列化，然后保存 params的一些安全子集：

class Notification < ActiveRecord::Base
  serialize :metadata, JSON
end

...

notification = Notification.new(
  metadata: params.permit(:SmsSid, :SmsStatus),
)

之后，您可以访问notification.params，它将为您透明地进行序列化。

请注意，只需将整个不动力的参数对象丢入数据库中，这是允许攻击者用千兆字节的垃圾文本淹没数据库的好方法。您绝不应该假设params是安全的；仅处理您期望的参数的子集，并确保关联的值符合您的期望。只需使用params.parmit（...），就像我上面所做的那样，仍然不够，因为值可能很长，或包含任意垃圾。您应该在模型上使用验证器来执行长度和格式限制。

Answer 2

我认为将数据存储为JSON字符串可以解决问题。这样：

class NotificationsController < ApplicationController
  def create
    notification = Notification.new(
      metadata: params.to_json,
    )
    if notification.save
      head :ok
    end
  end
end

然后：

JSON.parse(Notification.first.metadata)

或者，如果不想在任何地方写上述代码，您可以覆盖元数据getter方法

class Notification < ApplicationRecord

  def metadata
    JSON.parse(self[:metadata])
  end
end