加入收藏 | 设为首页 | 会员中心 | 我要投稿 李大同 (https://www.lidatong.com.cn/)- 科技、建站、经验、云计算、5G、大数据,站长网!
当前位置: 首页 > 百科 > 正文

ruby-on-rails-4 – 未受保护的重定向未被only_path修复

发布时间:2020-12-17 02:47:38 所属栏目:百科 来源:网络整理
导读:我有一个Rails 4应用程序,当我运行Brakeman时,它(正确地)在我的创建操作中标识了一个不受保护的重定向.但是,添加only_path:true(如在 Brakeman Railscast中)并不能解决警告: def create refer_url = params[:referrer] @portfolio = current_user.portfoli
我有一个Rails 4应用程序,当我运行Brakeman时,它(正确地)在我的创建操作中标识了一个不受保护的重定向.但是,添加only_path:true(如在 Brakeman Railscast中)并不能解决警告:

def create
    refer_url = params[:referrer]
    @portfolio = current_user.portfolios.build(portfolio_params)
    if @portfolio.save
      redirect_to refer_url,notice: "Portfolio was successfully created.",only_path: true
    else
      render :new
    end
  end

结果是:

+SECURITY WARNINGS+

+------------+-----------------------+---------+--------------+----------------------------------------------------------------------------------------------------------------------->>
| Confidence | Class                 | Method  | Warning Type | Message                                                                                                               >>
+------------+-----------------------+---------+--------------+----------------------------------------------------------------------------------------------------------------------->>
| High       | PortfoliosController  | create  | Redirect     | Possible unprotected redirect near line 14: redirect_to(+params[:referrer]+,:notice => "Portfolio was successfully cr>>
+------------+-----------------------+---------+--------------+----------------------------------------------------------------------------------------------------------------------->>

为什么会这样? Brakeman还在识别什么风险?

解决方法

不幸的是,RailsCast是不正确的. :only_path => true必须是第一个参数的一部分.

params [:referrer]应该是你申请中的路径吗?

如果是这样,这将是我的建议:

begin
  refer_url = URI.parse(params[:referrer]).path
rescue URI::InvalidURIError
  refer_url = "some_default"
end

或者您可以检查params [:referrer]始终是路径,以其他方式验证它,或者即使在您的应用程序中也不允许任意重定向.遗憾的是,Rails并没有为安全重定向提供简单的选择.

(编辑:李大同)

【声明】本站内容均来自网络,其相关言论仅代表作者个人观点,不代表本站立场。若无意侵犯到您的权利,请及时与联系站长删除相关内容!

    推荐文章
      热点阅读