javascript - 如何修复 JavaScript 中的开放重定向安全漏洞？

Question

Fortify Scan 报告

中的“Open Redirect”安全漏洞

window.open(strUrl,"_blank", features)

我的代码:

var features = "status=no, toolbar=no, menubar=no, location=no, top=20, left=175, height=500, width=750";

var strURL = ADMIN_TOOL_SNOOP + "?machineName=" + strMachine + "&template=snoop" +"&context=DEBUG&date=" + strDate.replace(regExp, "");

var validateStrURL =  new RegExp("/^[.\\p{Alnum}\\p{Space}]{0,1024}+/", "g");
if(validateStrURL.test(strURL))
{
  if((strURL != null)&&(strURL.length !=0)&&(strURL.length <= 200) )
  {
    window.open(strURL,"_blank",features);
  }
}

我该如何解决？

Answer 1

检测问题时，HP Fortify 会根据许多规则执行静态代码分析。在“打开重定向”问题的情况下，它检测到您正在重定向到一个 URL，该 URL 由传递给您的 JavaScript 或在您的 JavaScript 中创建的数据组成。这通常被认为容易受到网络钓鱼和其他攻击的操纵。

与其在客户端定义 ADMIN_TOOL_SNOOP 并使用它在 JavaScript 中构建 URL，不如尝试使用重定向页面在服务器端定义一组预先确定的目标，并且然后让您的 JavaScript 将目标页面作为参数传递。

例如，在您的客户端代码中有

window.open('redirect.php?destination=ADMIN_TOOL&template=...')

然后，在您的服务器端代码中，您将拥有一个类似于

的表格

DESTINATION_CODE | ACTUAL_URL
--------------------------------------
ADMIN_TOOL       | my_actual_page.php
ANOTHER_CODE     | another_page.php

当服务器收到重定向请求时，它应该查找匹配的代码并将页面重定向到实际的 URL。在代码不匹配的情况下，应该发生错误或不进行重定向。

总而言之，您正在创建一个由服务器端而非客户端控制的允许 URL 白名单，这意味着它不会对客户端操作开放。

HP reference for that scan result